WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 5 years ago

#12442 closed task (blessed) (fixed)

Change position of "Editor" in Appearance admin menu

Reported by: jane Owned by: dd32
Milestone: 3.0 Priority: normal
Severity: minor Version: 3.0
Component: UI Keywords: needs-patch
Focuses: Cc:

Description

Make Editor the last menu item under appearance, please. Thanks.

Attachments (1)

12442.diff (984 bytes) - added by markmcwilliams 6 years ago.

Download all attachments as: .zip

Change History (10)

@markmcwilliams6 years ago

comment:1 @markmcwilliams6 years ago

  • Keywords has-patch added

comment:2 @nacin6 years ago

  • Keywords needs-patch added; has-patch removed
  • Owner set to dd32
  • Status changed from new to assigned

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.

comment:3 @voyagerfan57616 years ago

  • Cc WordPress@… added

comment:4 @dd325 years ago

(In [13582]) Move 'Add New Themes' and 'Editor' to the bottom of the Appearance menu item. See #12442

comment:5 @dd325 years ago

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)

comment:6 @jane5 years ago

@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 @masonjames5 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!

comment:8 @dd325 years ago

@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.

comment:9 @dd325 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

Closing this ticket as its been implemented.

Note: See TracTickets for help on using tickets.