#19221 closed defect (bug) (fixed)
My Sites menu has invalid markup
Reported by: | wpmuguru | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.3 | Priority: | normal |
Severity: | normal | Version: | 3.3 |
Component: | Multisite | Keywords: | has-patch |
Focuses: | Cc: |
Description
The My Sites menu currently has a <ul>
within a <ul>
and no intervening <li>
when the user is a super admin.
Attachments (2)
Change History (7)
#1
@
13 years ago
The first patch fixed the issue for super admins but created an issue with non-super admins.
#2
@
13 years ago
Looks like 19221.2.diff has a logic inversion — we want sites to be secondary if there is a "Network Admin" menu, so secondary => true should be for super admins.
That said, [19230] forgot to force secondary items to primary items when there are no primary items. I'm going to commit a change for that as well.
Note: See
TracTickets for help on using
tickets.
ignore 19221.diff