Make WordPress Core

Opened 14 years ago

Closed 12 years ago

#15358 closed defect (bug) (worksforme)

The Network Admin > Users menu has a disclosue triangle even when there are no submenu items other than the Users page itself

Reported by: caesarsgrunt's profile caesarsgrunt Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: Network Admin Keywords:
Focuses: multisite Cc:

Description

The Network Admin > Users menu has a disclosue triangle even when there are no submenu items other than the Users page itself.

Attachments (1)

15358.diff (1.0 KB) - added by PeteMall 14 years ago.

Download all attachments as: .zip

Change History (11)

#1 @westi
14 years ago

  • Component changed from Administration to Network Admin
  • Milestone changed from Awaiting Review to 3.1

#2 @markmcwilliams
14 years ago

This was talked about in #wordpress-dev when [16183] and [16184] were made. We do have a /wp-admin/network/user-new.php file that should be linked also, but the only way to get them to appear, is to check the box on /wp-admin/network/settings.php that say Allow site administrators to add new users to their site via the "Users->Add New" page.

I don't know if this was still going to be looked into, but that should help you out! :)

#3 follow-up: @markmcwilliams
14 years ago

Related (if not a duplicate?): #15357

@PeteMall
14 years ago

#4 @PeteMall
14 years ago

  • Keywords has-patch added

#5 in reply to: ↑ 3 @caesarsgrunt
14 years ago

Replying to markmcwilliams:

Related (if not a duplicate?): #15357

Related but not a duplicate. This one is a bug in the menus system (I guess), whilst the other is querying the decision to base the ability of a super_admin to add users on an unrelated setting.

#6 follow-up: @PeteMall
14 years ago

  • Keywords needs-patch added; has-patch removed

We should add an extra check for submenus in wp-admin/includes/menu.php and remove the submenu if there is only one and it's the same as the top level menu.

#7 @ryan
14 years ago

  • Keywords 3.2-early added
  • Milestone changed from 3.1 to Future Release

#8 in reply to: ↑ 6 @nacin
12 years ago

Replying to PeteMall:

We should add an extra check for submenus in wp-admin/includes/menu.php and remove the submenu if there is only one and it's the same as the top level menu.

This code is already present for the regular admin. Should be fairly easy to apply to all three menus.

#9 @nacin
12 years ago

  • Keywords easy-fix added; 3.2-early removed

#10 @wonderboymusic
12 years ago

  • Keywords needs-patch easy-fix removed
  • Milestone Future Release deleted
  • Resolution set to worksforme
  • Status changed from new to closed

All of this stuff was fixed outside of this ticket

Note: See TracTickets for help on using tickets.