Closed
Bug 430540
Opened 16 years ago
Closed 14 years ago
Microformats in a floated div are not displayed (they are hidden)
Categories
(Toolkit Graveyard :: Microformats, defect)
Toolkit Graveyard
Microformats
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mkaply, Assigned: mkaply)
References
()
Details
Attachments
(2 files)
2.09 KB,
patch
|
sayrer
:
review+
mconnor
:
approval1.9.0.4-
|
Details | Diff | Splinter Review |
1.70 KB,
patch
|
sayrer
:
review+
|
Details | Diff | Splinter Review |
Currently in the microformats code I use the box object or bounding rect to determine if a microformat is visible on the screen or not. In cases where a microformat is marked as a float left or right and has only block level children, that microformat is marked invisible even though its children are visible. For the fix, I enumerate one level down to see if the children are visible. I tried going through all descendants, but that impacted performance. This should cover most cases.
Attachment #317359 -
Flags: review?(sayrer)
Comment 1•16 years ago
|
||
shouldn't we be able to test this? Patches at this point in the release cycle absolutely must have tests.
Assignee | ||
Comment 2•16 years ago
|
||
Here's the unit test
Assignee | ||
Comment 3•16 years ago
|
||
Sorry, I had simply forgotten to add the test.
Attachment #317366 -
Attachment is patch: true
Attachment #317366 -
Attachment mime type: application/octet-stream → text/plain
Assignee | ||
Comment 4•16 years ago
|
||
Comment on attachment 317366 [details] [diff] [review] Unit test Sorry, I didn't attach the unit test the first time.
Attachment #317366 -
Flags: review?(sayrer)
Updated•16 years ago
|
Attachment #317359 -
Flags: review?(sayrer) → review+
Updated•16 years ago
|
Attachment #317366 -
Flags: review?(sayrer) → review+
Assignee | ||
Comment 5•16 years ago
|
||
Fix checked in.
Assignee | ||
Comment 6•16 years ago
|
||
Comment on attachment 317359 [details] [diff] [review] Fix for problem - enumerate first level children's visibility Microformats correctness
Attachment #317359 -
Flags: approval1.9.0.3?
Comment 7•16 years ago
|
||
Comment on attachment 317359 [details] [diff] [review] Fix for problem - enumerate first level children's visibility In order to reduce risk and minimize overhead for stability releases, we are not accepting "nice to have" patches in stability releases. As 3.1 is fast approaching, this should not unduly impact time to get these fixes to users.
Attachment #317359 -
Flags: approval1.9.0.4? → approval1.9.0.4-
Assignee | ||
Comment 8•14 years ago
|
||
This was fixed on trunk.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•