[ICE-11118] ace:tooltip global attribute issue - ICEsoft JIRA Issue Tracker

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: EE-4.1.0.RC1
Fix Version/s: 4.2
Component/s: ACE-Components
Labels:
None
Environment:
ICEfaces4 trunk r.49103
Server: Tomcat7
Browsers: all

Assignee Priority:
P2

Description

QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/tooltip

ace:tooltip > DynamicAttribute:

The global attribute testing fails for the h:outputText and h:commandLink, the html title being rendered Instead of the global tooltip. This test passes only for the h:commandButton.

This can be reproduced with the steps:

- navigate to /tooltipDynAttribute.jsf.
- enable "rendered" and "global" attributes.
- mouse over "OutputText" or "ComandLink"; see how the html title attributes are rendered instead of the global tooltip (fail).
- mouse over "CommandButton", the global tooltip is rendered (pass).

This issue can be reproduced with ICEfaces EE-4.1.0.RC1, but not with ICEfaces 4.1.1.

This issue is a regression caused by ~~ICE-10893~~.

Activity

Ascending order - Click to sort in descending order

Carmen Cristurean created issue - 10/Aug/16 4:30 PM

Carmen Cristurean made changes - 10/Aug/16 4:30 PM

Field	Original Value	New Value
Fix Version/s		4.2 [ 12870 ]

Carmen Cristurean made changes - 11/Aug/16 9:27 AM

Description

QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/tooltip

ace:tooltip > DynamicAttribute:

The global attribute testing fails for the outputText and commandLink, the html title being rendered Instead of the global tooltip. This test passes only for the commandButton.

This can be reproduced with the steps:

- navigate to /tooltipDynAttribute.jsf.
- enable "rendered" and "global" attributes.
- mouse over "OutputText" or "ComandLink"; see how the html title attributes are rendered instead of the global tooltip (fail).
- mouse over "CommandButton", the global tooltip is rendered (pass).

QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/tooltip

ace:tooltip > DynamicAttribute:

The global attribute testing fails for the outputText and commandLink, the html title being rendered Instead of the global tooltip. This test passes only for the commandButton.

This can be reproduced with the steps:

- navigate to /tooltipDynAttribute.jsf.
- enable "rendered" and "global" attributes.
- mouse over "OutputText" or "ComandLink"; see how the html title attributes are rendered instead of the global tooltip (fail).
- mouse over "CommandButton", the global tooltip is rendered (pass).

This issue can be reproduced with ICEfaces EE-4.1.0.RC1, but not with ICEfaces 4.1.1.

Carmen Cristurean made changes - 11/Aug/16 10:10 AM

Description

QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/tooltip

ace:tooltip > DynamicAttribute:

The global attribute testing fails for the outputText and commandLink, the html title being rendered Instead of the global tooltip. This test passes only for the commandButton.

This can be reproduced with the steps:

- navigate to /tooltipDynAttribute.jsf.
- enable "rendered" and "global" attributes.
- mouse over "OutputText" or "ComandLink"; see how the html title attributes are rendered instead of the global tooltip (fail).
- mouse over "CommandButton", the global tooltip is rendered (pass).

This issue can be reproduced with ICEfaces EE-4.1.0.RC1, but not with ICEfaces 4.1.1.

QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/tooltip

ace:tooltip > DynamicAttribute:

The global attribute testing fails for the outputText and commandLink, the html title being rendered Instead of the global tooltip. This test passes only for the commandButton.

This can be reproduced with the steps:

- navigate to /tooltipDynAttribute.jsf.
- enable "rendered" and "global" attributes.
- mouse over "OutputText" or "ComandLink"; see how the html title attributes are rendered instead of the global tooltip (fail).
- mouse over "CommandButton", the global tooltip is rendered (pass).

This issue can be reproduced with ICEfaces EE-4.1.0.RC1, but not with ICEfaces 4.1.1.

This issue is a regression caused by ~~ICE-10893~~.

Judy Guglielmin made changes - 11/Aug/16 10:14 AM

Assignee

Arturo Zambrano [ artzambrano ]

Carmen Cristurean made changes - 12/Aug/16 9:54 AM

Description

QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/tooltip

ace:tooltip > DynamicAttribute:

The global attribute testing fails for the outputText and commandLink, the html title being rendered Instead of the global tooltip. This test passes only for the commandButton.

This can be reproduced with the steps:

- navigate to /tooltipDynAttribute.jsf.
- enable "rendered" and "global" attributes.
- mouse over "OutputText" or "ComandLink"; see how the html title attributes are rendered instead of the global tooltip (fail).
- mouse over "CommandButton", the global tooltip is rendered (pass).

This issue can be reproduced with ICEfaces EE-4.1.0.RC1, but not with ICEfaces 4.1.1.

This issue is a regression caused by ~~ICE-10893~~.

Ken Fyten made changes - 13/Dec/16 10:30 AM

Assignee

Arturo Zambrano [ artzambrano ]

Carmen Cristurean [ ccristurean ]

Hide

Permalink

Carmen Cristurean added a comment - 13/Dec/16 10:43 AM

This is still an issue with IF4 trunk r49677 (tested on FF49, MsEdge38, Chrome55).

Show

Carmen Cristurean added a comment - 13/Dec/16 10:43 AM This is still an issue with IF4 trunk r49677 (tested on FF49, MsEdge38, Chrome55).

Ken Fyten made changes - 13/Dec/16 10:45 AM

Assignee

Carmen Cristurean [ ccristurean ]

Arturo Zambrano [ artzambrano ]

Ken Fyten made changes - 13/Dec/16 10:45 AM

Assignee Priority

P2 [ 10011 ]

Hide

Permalink

Arturo Zambrano added a comment - 22/Dec/16 3:31 PM - edited

r49748: fix to assign separate internal IDs to individual qtip instances created with the 'global' attribute

The issue wasn't with specific components. The tooltip was only being applied to the last element that had a 'title' attribute, because having the same IDs was causing to replace the instances of the previous elements.

Show

Arturo Zambrano added a comment - 22/Dec/16 3:31 PM - edited r49748: fix to assign separate internal IDs to individual qtip instances created with the 'global' attribute The issue wasn't with specific components. The tooltip was only being applied to the last element that had a 'title' attribute, because having the same IDs was causing to replace the instances of the previous elements.

Arturo Zambrano made changes - 22/Dec/16 3:31 PM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Ken Fyten made changes - 22/Dec/16 3:41 PM

Security

Private [ 10001 ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#49748	Thu Dec 22 15:50:17 MST 2016	art.zambrano	~~ICE-11118~~ fix to assign separate IDs to individual qtip instances created with the 'global' attribute
				Files Changed
				MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/tooltip/tooltip.js

Arturo Zambrano made changes - 23/Dec/16 5:11 PM

Resolution	Fixed [ 1 ]
Status	Resolved [ 5 ]	Reopened [ 4 ]

Hide

Permalink

Liana Munroe added a comment - 02/Jan/17 11:33 AM

ICEfaces 4 trunk r49751, Tomcat 8
Showcase ace:tooltip > Global demo fails.
The tooltip on each element is rendered empty.

Show

Liana Munroe added a comment - 02/Jan/17 11:33 AM ICEfaces 4 trunk r49751, Tomcat 8 Showcase ace:tooltip > Global demo fails. The tooltip on each element is rendered empty.

Hide

Permalink

Arturo Zambrano added a comment - 03/Jan/17 3:26 PM

r49778: fixed issue with gobal tooltip displaying one more time after having been disabled; corrected global tooltip functionality to display the contents of the element's 'title' attribute if no content has been specified

Show

Arturo Zambrano added a comment - 03/Jan/17 3:26 PM r49778: fixed issue with gobal tooltip displaying one more time after having been disabled; corrected global tooltip functionality to display the contents of the element's 'title' attribute if no content has been specified

Arturo Zambrano made changes - 03/Jan/17 3:26 PM

Status	Reopened [ 4 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#49778	Tue Jan 03 15:45:58 MST 2017	art.zambrano	~~ICE-11118~~ fixed issue with gobal tooltip displaying one more time after having been disabled; corrected global tooltip functionality to display the contents of the element's 'title' attribute if no content has been specified
				Files Changed
				MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/tooltip/tooltip.js

Hide

Permalink

Liana Munroe added a comment - 03/Jan/17 4:13 PM

Verified ICEfaces 4 trunk r49778, Tomcat 8, Chrome 55, FF 47, MS Edge, IE 11, 10.
When toggling the global attribute in the tooltip dynamic attribute test a js error occurs when using IE 8.
Message: Object doesn't support this property or method
Line: 7
Char: 89
Code: 0
URI: http://10.18.39.134:8080/tooltip/javax.faces.resource/tooltip/tooltip.c.js.jsf?ln=icefaces.ace&v=4_2_0_170103
When doing the same using IE 11 emulating IE 8:
SCRIPT438: Object doesn't support property or method 'trim'
File: tooltip.c.js.jsf, Line: 7, Column: 116

Show

Liana Munroe added a comment - 03/Jan/17 4:13 PM Verified ICEfaces 4 trunk r49778, Tomcat 8, Chrome 55, FF 47, MS Edge, IE 11, 10. When toggling the global attribute in the tooltip dynamic attribute test a js error occurs when using IE 8. Message: Object doesn't support this property or method Line: 7 Char: 89 Code: 0 URI: http://10.18.39.134:8080/tooltip/javax.faces.resource/tooltip/tooltip.c.js.jsf?ln=icefaces.ace&v=4_2_0_170103 When doing the same using IE 11 emulating IE 8: SCRIPT438: Object doesn't support property or method 'trim' File: tooltip.c.js.jsf, Line: 7, Column: 116

Hide

Permalink

Ken Fyten added a comment - 03/Jan/17 4:17 PM

I think we can consider this a Known Issue on IE8 when dynamically changing the global attribute, since this is unlikely to be a pattern used in a real world application.

For future testing, reload the page once the global setting is made on IE8.

Show

Ken Fyten added a comment - 03/Jan/17 4:17 PM I think we can consider this a Known Issue on IE8 when dynamically changing the global attribute, since this is unlikely to be a pattern used in a real world application. For future testing, reload the page once the global setting is made on IE8.

Hide

Permalink

Arturo Zambrano added a comment - 03/Jan/17 5:39 PM

r49799: replaced 'trim' function for 'replace' function to trim string and avoid issues on ie8; expanded TLD documentation regarding global attribute to explain that the 'title' attribute used for the tooltip's content is overridden if explicit content is nested inside the component's tag

Show

Arturo Zambrano added a comment - 03/Jan/17 5:39 PM r49799: replaced 'trim' function for 'replace' function to trim string and avoid issues on ie8; expanded TLD documentation regarding global attribute to explain that the 'title' attribute used for the tooltip's content is overridden if explicit content is nested inside the component's tag

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#49779	Tue Jan 03 17:57:45 MST 2017	art.zambrano	~~ICE-11118~~ replaced 'trim' function for 'replace' function to trim string and avoid issues on ie8; expanded TLD documentation regarding global attribute
				Files Changed
				MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/tooltip/TooltipMeta.java MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/tooltip/tooltip.js

Hide

Permalink

Liana Munroe added a comment - 04/Jan/17 9:58 AM

Verified ICEfaces 4 trunk r49779, Tomcat 8, Chrome 55, FF 47, MS Edge, IE 11, 10, 8.

Show

Liana Munroe added a comment - 04/Jan/17 9:58 AM Verified ICEfaces 4 trunk r49779, Tomcat 8, Chrome 55, FF 47, MS Edge, IE 11, 10, 8.

Ken Fyten made changes - 13/Mar/17 1:33 PM

Status

Resolved [ 5 ]

Closed [ 6 ]

People

Assignee:

Arturo Zambrano

Reporter:

Carmen Cristurean

Votes:

0 Vote for this issue

Watchers:

4 Start watching this issue

Dates

Created:

10/Aug/16 4:30 PM

Updated:

13/Mar/17 1:33 PM

Resolved:

03/Jan/17 3:26 PM