Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: EE-4.1.0.GA, EE-3.3.0.GA_P04
-
Fix Version/s: 4.2, EE-3.3.0.GA_P05
-
Component/s: ACE-Components
-
Labels:None
-
Environment:IE, ACE menu components
-
Assignee Priority:P2
-
Support Case References:Support case #14018 - https://icesoft.my.salesforce.com/5007000001hGGUi
Description
This is reproducible with the Showcase demos.
-
Hide
- Case14018Example.war
- 10.10 MB
- Arran Mccullough
-
- META-INF/MANIFEST.MF 0.1 kB
- META-INF/context.xml 0.1 kB
- WEB-INF/classes/.../support/TestBean.class 0.9 kB
- WEB-INF/lib/icefaces-ace.jar 6.42 MB
- WEB-INF/lib/icefaces.jar 695 kB
- WEB-INF/lib/javax.faces-2.2.14.jar 3.00 MB
- WEB-INF/web.xml 2 kB
- index.xhtml 0.5 kB
- newWindow.xhtml 0.7 kB
- welcomeICEfaces.xhtml 1 kB
-
Hide
- Case14018Example.zip
- 21 kB
- Arran Mccullough
-
- Case14018Example/build.xml 3 kB
- Case14018Example/.../ant-deploy.xml 2 kB
- Case14018Example/.../build-impl.xml 80 kB
- Case14018Example/.../genfiles.properties 0.5 kB
- Case14018Example/.../private.properties 0.6 kB
- Case14018Example/nbproject/.../private.xml 0.3 kB
- Case14018Example/.../project.properties 4 kB
- Case14018Example/nbproject/project.xml 0.9 kB
- Case14018Example/src/conf/MANIFEST.MF 0.0 kB
- Case14018Example/src/.../TestBean.java 0.7 kB
- Case14018Example/web/index.xhtml 0.5 kB
- Case14018Example/web/.../context.xml 0.1 kB
- Case14018Example/web/newWindow.xhtml 0.7 kB
- Case14018Example/web/WEB-INF/web.xml 2 kB
- Case14018Example/.../welcomeICEfaces.xhtml 1 kB
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
r50536: fix for removing ui-state-focus styling when focusing out
This is actually a programmed behaviour to add the ui-state-focus style class to menu items. It just so happens to be triggered on right clicks as well, and the styling looks the same as the hover styling. This fix adds an event listener to remove such styling when focusing out.
r50537: Committed fix to the 3.3 EE maintenance branch.
I did a quick test with this fix and it does look like it resolves the right-click behavior but the issue with the sample app is stoll present. The test case is for a left-click which opens a new window.
r50545: fix for preventing focus styling on menu items that contain a call to open a new window (window.open())
Highlighting the last focused item was actually the way things are supposed to work. If you try the ace:menuBar Overview demo and click on the 'Home' menu item, it will open a new window, and it will be highlighted in order to be able to tab through the rest of the menu items. Clicking anywhere else on the page removes this styling.
I tried to prevent this behaviour anyway in many obvious ways and non-obvious ways, but it was impossible to do it, because the sudden change to a new window makes it hard to keep track of the original event, and because the event used (focusin) is a synthetic event, there's not enough information about how the even really started. So, I went for a brute force approach to prevent this focus styling on items that open new windows via the window.open() function. That's the only solution unless we want to spend more time making major modifications to the Wijmo code. I hope it solves the customer request.
Can this fix be added to the 3.3.0 maintenance branch as well?
r50553: applied latest fix to the 3.3 maintenance branch
Attached test case that reproduces the issue in a different way.
Steps: