Opened 3 years ago
Closed 3 years ago
#12442 closed task (blessed) (fixed)
Change position of "Editor" in Appearance admin menu
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.0 |
| Component: | UI | Version: | 3.0 |
| Severity: | minor | Keywords: | needs-patch |
| Cc: | WordPress@… |
Description
Make Editor the last menu item under appearance, please. Thanks.
Attachments (1)
Change History (10)
markmcwilliams — 3 years ago
comment:1
markmcwilliams — 3 years ago
- Keywords has-patch added
- Keywords needs-patch added; has-patch removed
- Owner set to dd32
- Status changed from new to assigned
comment:3
voyagerfan5761 — 3 years ago
- Cc WordPress@… added
I'm open to that function being renamed or moved elsewhere, which is why i've left this open for now.
As nacin said, I'd hit this before, thus, didnt notice the Add New themes item had been removed already.. See [13583] which removed it again.
Priority is set to 101 to hook late after all plugins should've run (I cant think of a reason a plugin would hook to 100, but just in case)
@dd32: "Add New Themes" shouldn't be on the menu at all any more, that's the point of the tabs on Themes.
comment:7
masonjames — 3 years ago
I posted a ticket on this, but maybe it's better served here. The "background" menu option under Appearance seems to break the sidebar widget toggle ability. Selecting any other menu item brings back the toggle as expected. Thanks!
@dd32: "Add New Themes" shouldn't be on the menu at all any more
I hadnt noticed that it had been removed, as it was still in my file. Thats the reason for the followup commit which reverted that.
Unfortunately, as dd32 was investigating this morning re: 'Add New Themes', pushing these menu items around can get complicated, especially when we then add the 'Background' and 'Header' items.
I'll assign this to him as he had come up with a solution, I just don't think he patched it yet.