Issue Details (XML | Word | Printable)

Key: FLAMINGO-70
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: kirillcool
Reporter: krzych_nowak
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
flamingo

Setting popup rich tooltip to null hides the action tooltip

Created: 30/Mar/10 05:18 AM   Updated: 31/Mar/10 04:21 PM   Resolved: 31/Mar/10 04:21 PM
Component/s: ribbon
Affects Version/s: 5.0
Fix Version/s: milestone 1

Time Tracking:
Not Specified

Environment:

Operating System: All
Platform: All


Issuezilla Id: 70
Tags:
Participants: kirillcool and krzych_nowak


 Description  « Hide

When setting the popup rich tooltip to null, commandbutton is unregistered from
RichTooltipManager.
Current implementation does not check if the commandbutton action rich tooltip
is != null, it just unregisters the component.
Expected behaviour is that modifing popup tooltip will not modify the action tooltip

Current impl:
this.popupRichTooltip = richTooltip;
RichToolTipManager richToolTipManager = RichToolTipManager
.sharedInstance();
if (richTooltip != null) { richToolTipManager.registerComponent(this); } else { richToolTipManager.unregisterComponent(this); }
Should be replaced with something like:
this.popupRichTooltip = richTooltip;
RichToolTipManager richToolTipManager = RichToolTipManager
.sharedInstance();
if (hasRichTooltip()) { // tests if the component has any tooltip to display richToolTipManager.registerComponent(this); } else { richToolTipManager.unregisterComponent(this); } }

Also the popupRichTooltip, and actionRichTooltip cant be obtained as they are
private and there is no getters. They could be protected (to allow for example
disabled tooltips)

Krzysztof



kirillcool added a comment - 31/Mar/10 04:21 PM

This issue should be fixed in the latest 5.0dev drop. If this still happens,
please attach a small (<100 lines) test application that reproduces this
behavior under the latest 5.0dev drop.

The second part is more of an RFE - please do not mix two different issues in
one bug report. If you want to discuss this, the forum is a better place -
especially to describe what do you mean by the "disabled tooltips".

Also, please do not reply to the automated e-mail and keep the discussion of the
original bug in the bug tracker so that its history is not lost.

Thanks
Kirill