CSS errors

Bug #426531 reported by Michael Bienia
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Deryck Hodge

Bug Description

This is on edge (r9365).

One CSS error is in the inline CSS stylesheet in e.g. https://bugs.edge.launchpad.net/launchpadlib/+bug/325367:

      <style type="text/css">
        /* A page-specific fix for inline text are editing to line up box. */
        #edit-description .yui-ieditor-input { top: 0; }
        /* Align the 'add comment' link to the right of the comment box. */
        #add-comment-form textarea { width: 100%; max-width=60em;}
        #add-comment-form { max-width: 60em; padding-bottom: 4em; }
        #add-comment-form .actions {float: right;}
      </style>

There is one "=" instead ":" in one CSS declaration.

An other error is in https://edge.launchpad.net/+icing/rev9365/lazr/build/choiceedit/assets/choiceedit-core.css:
     .yui-ichoicelist #yui-pretty-overlay-modal h2 {/
The "/" doesn't belong there.

Firefox complains about many more errors in the CSS files. Some seem to be missing newlines (https://edge.launchpad.net/+icing/rev9365/+style-slimmer.css or https://edge.launchpad.net/+icing/rev9365/lazr/build/autocomplete/assets/skins/sam/autocomplete.css) causing parse errors. It would be probably a good idea to check them all with a validator.

Related branches

Michael Bienia (geser)
affects: launchpad → launchpad-foundations
Revision history for this message
Curtis Hovey (sinzui) wrote :

I don't think the broken instruction is necessary, all <textarea>s are set to 60em

Changed in launchpad-foundations:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Curtis Hovey (sinzui) wrote :

We do not have any control over the YUI CSS, but the bugs team can fix the errors in their markup, or just delete the rules since they are broken and thus not being used.

affects: launchpad-foundations → malone
tags: added: post-3-ui-cleanup
Deryck Hodge (deryck)
Changed in malone:
assignee: nobody → Deryck Hodge (deryck)
status: Triaged → In Progress
Revision history for this message
Diogo Matsubara (matsubara) wrote : Bug fixed by a commit
Changed in malone:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
tags: added: css
Changed in malone:
status: Fix Committed → Fix Released
Deryck Hodge (deryck)
Changed in malone:
milestone: none → 3.1.12
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.