Page MenuHomePhabricator

Lets not have "Delete" option enabled when the only item inside Transclusion Dialog is "Add template" , as that cannot be deleted
Closed, ResolvedPublic8 Estimated Story Points

Description

Recently we tweaked the behavior inside the tranclusion dialog in a way that user wont get stranded by deleting everything inside the dialog.It will always have the "Add a template" textbox prompted.(https://phabricator.wikimedia.org/T52281)
Now we still have the trash icon on both the left and right panel enabled, clicking on which does nothing, as you cannot empty the dialog.
So can we just disable the Trash button here?

Screen_Shot_2015-04-06_at_5.00.49_PM.png (508×701 px, 34 KB)

Event Timeline

Ryasmeen raised the priority of this task from to Needs Triage.
Ryasmeen updated the task description. (Show Details)
Ryasmeen added a project: VisualEditor.
Ryasmeen subscribed.
Ryasmeen set Security to None.

Also trash icon is red on the right side panel and black on the left side

Change 226975 had a related patch set uploaded (by Alex Monk):
Stop removal of only template placeholder from transclusion

https://gerrit.wikimedia.org/r/226975

Jdforrester-WMF removed a project: Patch-For-Review.
Jdforrester-WMF edited a custom field.

Change 226975 merged by jenkins-bot:
Stop removal of only template placeholder from transclusion

https://gerrit.wikimedia.org/r/226975

We still have the red trash icon on the right panel, do we want it like this?

Screen Shot 2015-08-05 at 10.55.53 PM.png (504×701 px, 31 KB)

Change 230814 had a related patch set uploaded (by Alex Monk):
Template placeholder: Don't show delete button if we're the only transclusion part

https://gerrit.wikimedia.org/r/230814

Change 230814 merged by jenkins-bot:
Template placeholder: Don't show delete button if we're the only transclusion part

https://gerrit.wikimedia.org/r/230814