Closed
Bug 571842
Opened 15 years ago
Closed 15 years ago
add classes to split menu for CSS addressability
Categories
(Firefox :: Menus, defect)
Tracking
()
RESOLVED
FIXED
Firefox 4.0b1
People
(Reporter: mcdavis941.bugs, Assigned: mcdavis941.bugs)
References
Details
Attachments
(1 file)
(deleted),
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
Dao, would it be possible to add some more classes added to the new split menu for addressability in CSS? Something like:
<menupopup>
...
<hbox class="split-menuitem">
<menuitem class="split-menuitem-item/>
<menu class="split-menuitem-menu"/>
</hbox>
...
</menupopup>
or whatever you think is appropriate. The key point being adding something to the hbox container and to the contained menuitem.
Assignee | ||
Updated•15 years ago
|
Blocks: FirefoxButton
Comment 1•15 years ago
|
||
Sure, would you like to write the patch?
Assignee | ||
Comment 2•15 years ago
|
||
(In reply to comment #1)
> Sure, would you like to write the patch?
Thanks, that's a nice offer. Unfortunately, I checked my build environment and I need to update .. all of it. (Source tree, sdk, and tools.) It might only be a day or two before that's done, or it could end up being several days by the time all the wrinkles are ironed out.
In the mean time, there's not much to this bug, so if you want me to provide edited but untested files (or file) I could do that.
Either way, I'll check back in when I can build and test, and if not this time then maybe later.
Assignee | ||
Comment 3•15 years ago
|
||
Very tiny patch.
If someone wants to suggest class names that better conform to naming conventions, I'll give it another go, but this seems reasonable.
I'm still running tests; I'll ask for review when that's done.
Assignee | ||
Comment 4•15 years ago
|
||
Comment on attachment 452861 [details] [diff] [review]
wip patch - adds classes
Asking Dao for review of the patch in comment 3.
Attachment #452861 -
Flags: review?(dao)
Updated•15 years ago
|
Attachment #452861 -
Flags: review?(dao) → review+
Updated•15 years ago
|
Assignee: nobody → mcdavis941.bugs
Keywords: checkin-needed
Comment 5•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a6
Is there any chance to implement the new split menu in XBL (e.g. <menu type="split">)?
Current hbox + menuitem + menu implementation isn't versatile for extensions.
Comment 7•14 years ago
|
||
(In reply to comment #6)
> Is there any chance to implement the new split menu in XBL (e.g. <menu
> type="split">)?
> Current hbox + menuitem + menu implementation isn't versatile for extensions.
You should file a bug on it, first of all. The chance that it will happen will also dramatically increase if you contribute a patch. ;-)
Comment 8•14 years ago
|
||
(In reply to comment #6)
> Is there any chance to implement the new split menu in XBL (e.g. <menu
> type="split">)?
> Current hbox + menuitem + menu implementation isn't versatile for extensions.
filed bug 613156
You need to log in
before you can comment on or make changes to this bug.
Description
•