Closed Bug 430540 Opened 12 years ago Closed 10 years ago
Microformats in a floated div are not displayed (they are hidden)
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)
shouldn't we be able to test this? Patches at this point in the release cycle absolutely must have tests.
Here's the unit test
Sorry, I had simply forgotten to add the test.
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)
Fix checked in.
Comment on attachment 317359 [details] [diff] [review] Fix for problem - enumerate first level children's visibility Microformats correctness
Attachment #317359 - Flags: approval184.108.40.206?
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: approval220.127.116.11? → approval18.104.22.168-
This was fixed on trunk.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.