As the title says.
Version: unspecified
Severity: minor
Description
Description
Details
Details
- Reference
- bz40813
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Declined | None | T42813 mw.util.addPortletLink doesn't work for third party skins that have different portlet HTML | |||
| Resolved | Krinkle | T42814 Standardise portlet html structure |
Event Timeline
Comment Actions
Should it?
The only thing addPortletLink needs to work properly is an <ul> inside the portlet element (and it still sort of works if that's missing). This is a pretty reasonable expectation and the usual convention for menus all over the Internet. Maybe it should support the portlet being an <ul> by itself - and that should be it.
Comment Actions
There is a very minimal HTML requirement for skins to make this work. Adding hooks for these or other abstractions is unnecessary.
Note that it is already supported to not have any portlets (such as in Minerva), in which case addPortletLink() does not cause an error.