PrettyOverlay removes TD border from content

Bug #570363 reported by Sidnei da Silva
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LAZR Javascript Library
Fix Released
Undecided
Paul Hummer

Bug Description

Adding a table inside the PrettyOverlay clears all the table borders. I had to make the following change to work around it:

=== modified file 'src/canonical/landscape/static/yui/lazr/overlay/assets/pretty-overlay-core.css'
--- src/canonical/landscape/static/yui/lazr/overlay/assets/pretty-overlay-core.css 2010-04-06 19:32:23 +0000
+++ src/canonical/landscape/static/yui/lazr/overlay/assets/pretty-overlay-core.css 2010-04-26 19:04:17 +0000
@@ -21,8 +21,16 @@
     text-align: left;
 }

-/* Ensure that td has no border (YUI base css adds one). */
-.yui3-pretty-overlay td {
+/* Ensure that the shadow td's have no border (YUI base css adds one). */
+.yui3-pretty-overlay .ne,
+.yui3-pretty-overlay .north,
+.yui3-pretty-overlay .nw,
+.yui3-pretty-overlay .east,
+.yui3-pretty-overlay .center,
+.yui3-pretty-overlay .west,
+.yui3-pretty-overlay .se,
+.yui3-pretty-overlay .south,
+.yui3-pretty-overlay .sw {
     border-width: 0;
     padding: 0;
 }

=== modified file 'src/canonical/landscape/static/yui/lazr/overlay/overlay.js'
--- src/canonical/landscape/static/yui/lazr/overlay/overlay.js 2010-04-13 13:11:56 +0000
+++ src/canonical/landscape/static/yui/lazr/overlay/overlay.js 2010-04-26 19:04:17 +0000
@@ -349,7 +349,7 @@
         '<tr>',
         '<td class="east">',
         '</td>',
- '<td>',
+ '<td class="center">',
         '<div style="border: 1px solid white" id="yui3-pretty-overlay-modal" ',
         ' class="content_box_container">',
         '<div class="close">',

Revision history for this message
Paul Hummer (rockstar) wrote :

Sidnei, is this stil occurring after my changes? I suspect the answer is no, but I thought I would ask first.

Changed in lazr-js:
status: New → Incomplete
importance: Medium → Undecided
Revision history for this message
Paul Hummer (rockstar) wrote :

I just verified this is no longer occurring.

Changed in lazr-js:
status: Incomplete → Fix Released
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.