Opened 12 years ago
Closed 11 years ago
#8983 closed defect (fixed)
MenuBar borderContainer classes risky
Reported by: | dante | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 1.5 |
Component: | Dijit - LnF | Version: | 1.3.0b3 |
Keywords: | Cc: | bill, nonken | |
Blocked By: | Blocking: |
Description
For some reason MenuBar isn't getting the class ".dijitBorderContainer-dijitMenuBar" set when it is a child of a borderContainer. Adding a MenuBar to themeTester exposed this. This is to track the fix for the class (baseClass related?)
Going to fix for the non-embedded case, and add to themeTester in 1.3 as part of #8885, but need to address the larger issue.
Change History (4)
comment:1 Changed 12 years ago by
comment:2 Changed 11 years ago by
Milestone: | 1.4 → 1.5 |
---|
comment:3 Changed 11 years ago by
Owner: | changed from dante to bill |
---|---|
Status: | new → assigned |
Right, Menu is missing baseClass.
Incidentally, I had actually planned to add baseClass and convert Menu to use _CssStateMixin, but Menu has a weird usage of the word "active": dijitMenuActive refers not to a mouse-down event on the menu but to the menu having been focused, after which merely hovering a MenuItem will open it.
Anyway I'll add baseClass.
comment:4 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
started in [17124]