Make WordPress Core

Opened 12 years ago

Closed 12 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:


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

Attachments (1)

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

Download all attachments as: .zip

Change History (10)

#1 @markmcwilliams
12 years ago

  • Keywords has-patch added

#2 @nacin
12 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.

#3 @voyagerfan5761
12 years ago

  • Cc WordPress@… added

#4 @dd32
12 years ago

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

#5 @dd32
12 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)

#6 @jane
12 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.

#7 @masonjames
12 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!

#8 @dd32
12 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.

#9 @dd32
12 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.