WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 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 4 years ago.

Download all attachments as: .zip

Change History (10)

markmcwilliams4 years ago

comment:1 markmcwilliams4 years ago

  • Keywords has-patch added

comment:2 nacin4 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 voyagerfan57614 years ago

  • Cc WordPress@… added

comment:4 dd324 years ago

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

comment:5 dd324 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 jane4 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 masonjames4 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 dd324 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 dd324 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.