#19245 closed defect (bug) (fixed)
Maintain back-compat for admin bar 'appearance'
Reported by: | nacin | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Toolbar | Keywords: | has-patch |
Focuses: | Cc: |
Description
We removed the 'appearance' menu in 3.3. It's fair to say that other plugins may have added nodes there.
What I think we should do is not only offer a parent translation, but to also keep the existing items under appearance
. It feels cleaner overall, and reduces the risk of clashing with plugin modifications. Thoughts?
Attachments (1)
Change History (10)
#5
@
13 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In [19365]:
#6
@
13 years ago
Now I remember why I considered making 'appearance' canonical. If we ever want to incorporate the entire admin menu, we can simply scrap the appearance items all together and render it automatically. Now, we may have additional submenu items added by plugins floating next to the top-level items.
Note: See
TracTickets for help on using
tickets.
On the other hand, plugins can continue adding to the old id for < 3.3 compat and use the new one for 3.3+ compat and everything will be nicely targeted. I think admin bar menu back compat can be much more relaxed than we are with the admin menus.