Details
-
Type: Bug
-
Status: Closed
-
Priority: Minor
-
Resolution: Fixed
-
Affects Version/s: 3.0.RC2
-
Fix Version/s: 3.0
-
Component/s: ACE-Components
-
Labels:None
-
Environment:ICEfaces 3, ace:subMenu
-
Assignee Priority:P1
Description
The ace:subMenu component is lacking border styling (no borders are rendered). This is viewable in the showcase sample application using the menuBar demo.
Not sure if this is just a theme styling issue, or a component issue.
Not sure if this is just a theme styling issue, or a component issue.
I remember this just wasn't playing nice with Themeroller themes. It was overriding a lot of the styling, and there wasn't anything else that could be done, because this class was being added dynamically. I made sure removing it didn't affect anything else, so I removed it. It has been working fine for more than a year.
In the current Wijmo code only checks for this class (i.e. hasClass()) on the root menu, not the submenus, and the only purpose of this is to add that class if it wasn't there. So, if the new code doesn't check for this class somewhere else, then there's no need to change this fix.