Closed
Bug 406790
Opened 17 years ago
Closed 17 years ago
Add support for an option to show/hide hidden microformats
Categories
(Toolkit Graveyard :: Microformats, defect)
Toolkit Graveyard
Microformats
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mkaply, Assigned: mkaply)
Details
Attachments
(2 files)
5.55 KB,
patch
|
benjamin
:
review+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
1.86 KB,
patch
|
Details | Diff | Splinter Review |
I'm adding a new option so that the developer can decide if they want to show or hide hidden microformats (display:none) This requires changing recurseFrames to an options JS structure (which I should have done in the first place)
Attachment #291442 -
Flags: review?(sayrer)
Assignee | ||
Updated•17 years ago
|
Attachment #291442 -
Flags: review?(sayrer) → review?(benjamin)
Comment 1•17 years ago
|
||
Comment on attachment 291442 [details] [diff] [review] Fix for problem This tentatively looks ok, but it's not approved to land without a unit-test.
Attachment #291442 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 2•17 years ago
|
||
Unit test and fixes to previous tests because of this functionality.
Assignee | ||
Comment 3•17 years ago
|
||
Comment on attachment 291442 [details] [diff] [review] Fix for problem This is a small update to the microformats API with unit test
Attachment #291442 -
Flags: approval1.9?
Updated•17 years ago
|
Attachment #291442 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 4•17 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 17 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
•