- Status Closed
- Percent Complete
- Task Type Bug Report
- Category Backend/Core
- Assigned To No-one
- Operating System All
- Severity Low
- Priority Medium
- Reported Version 1.0 alpha1
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#2037 - Task closing reason not mouse selectable
Probably javascript event handling race condition, see screenshot.
Show/hide click event listener from prototype.js probably. (I'm not js debugging expert)
Tested:
not working with:
- Firefox 40.0 on openSuSE13.2
working with:
- Firefox 40 on window 7
- Chromium on openSuSE13.2
Workaround:
- Click reason selector with mouse
- navigate with down/up keys to the wanted closing reason
- hit Enter key to choose the closing reason
Closed by peterdd
27.08.2015 22:48
Reason for closing: Fixed in devel
Additional comments about closing:
27.08.2015 22:48
Reason for closing: Fixed in devel
Additional comments about closing:
.. and in Mozilla Firefox for similiar
cases with version 40.0.3.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Encountered the same thing, a co-worker noticed about it. Seems that something has changed in Firefox, it was working properly in last spring. Did some testing and it still works with IE 11 (Windows 7), Edge (Windows 10), and the remaining on Kubuntu 15.04: Chrome 45, Opera (can't remember version number, the latest one, don't have it installed home) and Rekonq 2.4.2. Have to test what happens if I comment out line 5466 in prototype.js.
Is failing in firefox and Alpha 2
Something to do with onmouseup="Event.stop(event);" ??
Yea, I think the same.
I wanted replace the key listener event and replace it by pure css toggle for hide/show of closing form, but got a veto from psycho. Maybe I can convince him this time?
Maybe to emulate his 'click everywhere to close the popup closing form' by a underlying fullpage label..
The problem definitely is Event.stop(event). Here's the definition of it (in prototype.js):
Firefox as the only browser now actually implements preventDefault() correctly (in my opinion), so it prevents the selection from happening. Did some testing with a few different browsers again (Edge omitted, don't have it available here at home), and it seems that replacing it with onmouseup="event.stopPropagation();" should be enough to prevent the event bubbling upward also to project selector. I think that was original problem that the onmouseup event handler tried to prevent (it really happens if the event handler is removed). But waiting for a comment before making a pull request.
We were not the only ones with this problem.
https://bugzilla.mozilla.org/buglist.cgi?bug_id=1194316,1195675,1196390,1196486,1197347,1197533,1197655
But it is now fixed by Jouni in Flyspray source, and in Mozilla Firefox with Version 40.0.3..