![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#19221 closed defect (bug) (fixed)
My Sites menu has invalid markup
| Reported by: | wpmuguru | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.3 |
| Component: | Multisite | Version: | 3.3 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
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
@
15 years ago
The first patch fixed the issue for super admins but created an issue with non-super admins.
#2
@
15 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