Bug 733487 - Search bar saved searches menu ignores clicks
Summary: Search bar saved searches menu ignores clicks
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.0.1
Hardware: All
OS: All
medium
low vote
Target Milestone: ---
: ---
Assignee: Mike Thompson
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks: rhq41-ui rhq-uxd
TreeView+ depends on / blocked
 
Reported: 2011-08-25 19:28 UTC by Jay Shaughnessy
Modified: 2013-09-01 19:23 UTC (History)
2 users (show)

Fixed In Version: 4.5.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-01 19:23:00 UTC


Attachments (Terms of Use)


Links
System ID Priority Status Summary Last Updated
Red Hat Bugzilla 695889 None None None Never

Internal Links: 695889

Description Jay Shaughnessy 2011-08-25 19:28:12 UTC
Search bar saved searches menu ignores clicks

1) Create at least two saved search for the search bar, A and B.

2) Open the menu (click the the down arrow in the menu bar)

3) click on A to set it as the current search

4) Again open the menu (click the the down arrow in the menu bar)

5) click on B to set it as the current search

The click is ignored and the menu disappears.


Workaround: Open the menu again and it will work on the second try.

Comment 1 Jay Shaughnessy 2011-08-25 19:48:57 UTC
I've tried to get around this several ways but smartgwt seems to be 
eating the onClick event.  Note that onMouseOver/onMouseOut events are 
reported successfully.  It seems only that onClick is never invoked. 

This problem is almost definitely due to the use of GWT widgets in a
SmartGWT app. There are several mentions in the SmartGWT forums to
avoid what we're doing, and specifically one entry complaining of the
same situation:

http://forums.smartclient.com/showthread.php?t=6484&highlight=PopupPanel+onClick

I don't think there is a solution keeping with our current
implementation.  The solution would be to re-implement the SearchBar 
without PopupPanels, and use SmartGwt windows.

The workaround is easy, the next time the window opens it will work
as expected. It seems to eat every second click.

Comment 2 Mike Thompson 2012-07-31 01:06:08 UTC
Master commit: be31334

This turned out being a replacement of the GWT search bar with a smartGWT version that avoids the mixing weirdness associated with GWT and smartGWT as outlined above. Should be fully tested as it is a rewrite of the old search bar.

Comment 3 Heiko W. Rupp 2013-09-01 19:23:00 UTC
Bulk closing of BZs that have no target version set, but which are ON_QA for more than a year and thus are in production for a long time.


Note You need to log in before you can comment on or make changes to this bug.