https://bugzilla.wikimedia.org/show_bug.cgi?id=19704
Brion Vibber <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] Depends on| |11746 --- Comment #3 from Brion Vibber <[email protected]> 2009-07-19 20:49:22 UTC --- There are several issues rolled up into this "bug": 1) Dislike for use of document.write() based on rumor of instability: There's no ambiguity in behavior when this is used by synchronously-loaded scripts as in current CentralNotice. Only browser reported to have problems is a 6-year old pre-alpha of Mozilla Firebird, and can be quite freshly ignored. 2) The text is also inserted via document.write(), which there's no other general complaint about here... There is however in bug 11746 which appears to have been (mistakenly?) resolved as FIXED. Adding bug 11746 as a dependency; document.write() is incompatible with the wiki being served out as application/xhtml+xml. 3) Request not to append styles unnecessarily when the item is dismissed. Collapse/expand or dissmiss/show behavior in CentralNotice are actually *produced* via CSS; there's no inherent notion of a dismissed CentralNotice. (In fact it's highly discouraged; all messages should have a way to return to them, which is why it's designed to support collapsing/expanding primarily.) Not showing the CSS would simply result in the message being always shown. 4) The insertion of <style> into the document body instead of the header where it belongs in theory. This wasn't specifically mentioned as a reason, but would be resolved if doing an appendCss() call as requested. Note however that there are two style blocks relevant: 4a) The setup for hiding/showing of components based on cookie state 4b) Notice campaign-specific CSS, defined as a <style> chunk in the campaign template The latter is not produced by the software, and would continue to be inserted via document.write() (or else via something like innerHTML, but still in the wrong place) unless CentralNotice were changed to have separate HTML body and style sections for each campaign notice. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Wikibugs-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
