#2217 closed defect (fixed)
DropdownButton menu non-functional
Reported by: | Owned by: | Douglas Hays | |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | Widgets | Version: | 0.4.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description (last modified by )
Clicking on the menu item fails to activate the alert (or onClick event).
The strange thing is that the bug code is basically identical to the test code in tests/widget/test_Button.html which functions just fine.
The code still works fine in 0.3.1 for all browsers.
Attachments (2)
Change History (10)
Changed 14 years ago by
Attachment: | 3828.2.html added |
---|
comment:1 Changed 14 years ago by
Description: | modified (diff) |
---|---|
Milestone: | → 0.5 |
comment:2 Changed 14 years ago by
Owner: | changed from bill to Douglas Hays |
---|
Passing to Doug since he's modifying Button right now. The best thing I can think of is to put a timeout in Button.onBlur(), to delay closing the menu. That leaves open the issue of, when the menu closes, where is the focus? Should it go back to the button?
comment:4 Changed 14 years ago by
Becky says that using toggle="wipe" on the menu will work around the problem.
comment:5 Changed 14 years ago by
Thank-you to you & Becky. It did the trick.
As it turns out, it was the one toggle animation I didn't try.
comment:6 Changed 14 years ago by
Milestone: | 0.9 → 0.4.2 |
---|
Changed 14 years ago by
Attachment: | 2217.patch added |
---|
comment:7 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Ah, you should attach testcases to the bug; that's standard procedure. You probably didn't know you can do that because you can't attach until after you've already filed the bug. It's confusing.
Here's the issue:
Button.onBlur() needs to check if the blur event occured within the menu.