Page MenuHomePhabricator

Book editor panel must not use #siteNotice wrapper
Open, LowPublic

Description

Book editor panel must not use #siteNotice wrapper.

  • It semantically does not belong there. #siteNotice is for site notices. Not for other things. Create new sibling after #siteNotice instead.
  • Lot of people turn off all notices by using display:none for #siteNotice which disables the panel.
  • Such wrapper has random styling on local wikis which may override default settings of the panel.

Version: unspecified
Severity: enhancement

Details

Reference
bz24972

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 11:08 PM
bzimport added a project: Collection.
bzimport set Reference to bz24972.
bzimport added a subscriber: Unknown Object (MLST).
Danny_B created this task.Aug 28 2010, 8:53 PM

johannes.beigel wrote:

We're actually using the hook SiteNoticeAfter to render that box, so the #siteNotice ID is inserted by core MediaWiki code. Any ideas what we could use instead?

roland wrote:

This also causes trouble with http://www.mediawiki.org/wiki/Extension:DismissableSiteNotice
If both are enabled and the current SiteNotice was disabled per click, the book editor panel is completely missing.

roland wrote:

Including DismissableSiteNotice before Collection works around the problem for these both extensions, but doesn't solve the main problem.