Closed
Bug 307564
Opened 19 years ago
Closed 19 years ago
Richlistbox blinks when opening the context menu of a selected item
Categories
(Toolkit :: UI Widgets, defect)
Toolkit
UI Widgets
Tracking
()
RESOLVED
FIXED
People
(Reporter: asaf, Assigned: doronr)
Details
(Keywords: fixed1.8)
Attachments
(1 file)
|
941 bytes,
patch
|
robert.strong.bugs
:
first-review+
asa
:
approval1.8b5+
|
Details | Diff | Splinter Review |
Richlistbox blinks when opening the context menu of a selected item. STR: 1. select an item in the EM 2. Right click the same item
| Assignee | ||
Comment 1•19 years ago
|
||
make sure we don't redo selection if we are reselecting the same item. Could fix it in the event handlers, but a central place makes sense.
Attachment #195318 -
Flags: first-review?(rob_strong)
Comment 2•19 years ago
|
||
Comment on attachment 195318 [details] [diff] [review] le patch Now this I can handle ;) Thanks!
Attachment #195318 -
Flags: first-review?(rob_strong) → first-review+
Updated•19 years ago
|
OS: MacOS X → All
Hardware: Macintosh → All
| Assignee | ||
Updated•19 years ago
|
Attachment #195318 -
Flags: approval1.8b5?
| Assignee | ||
Comment 3•19 years ago
|
||
checked into trunk
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Comment 4•19 years ago
|
||
I don't see this in the latest branch build on win32. Is this really a problem?
| Assignee | ||
Comment 5•19 years ago
|
||
I can see it on linux, windows might be fast enough to avoid it. The patch is safe though, since it reduces the ammount of work in an case where no work needs to be done.
Comment 6•19 years ago
|
||
Comment on attachment 195318 [details] [diff] [review] le patch OK. Yes, I see this in Linux. approving.
Attachment #195318 -
Flags: approval1.8b5? → approval1.8b5+
Comment 8•19 years ago
|
||
Thanks, this also fixes the blinking when double clicking on a selected entry to open the associated options dialog.
You need to log in
before you can comment on or make changes to this bug.
Description
•