Ticket #3251 (closed defect: fixed)
Quick view doesn't grab focus on mouse click
Reported by: | egmont | Owned by: | andrew_b |
---|---|---|---|
Priority: | minor | Milestone: | 4.8.13 |
Component: | mc-core | Version: | 4.8.12 |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Branch state: | merged | Votes for changeset: | committed-master |
Description
Choose "Quick view" in the right panel.
Press Tab. The viewer gets the focus, subsequent arrow up/down keys scroll the file. The bottom button bar is also updated accordingly. Press Tab again to move the focus back to the file listing panel. Everything works as expected.
Click with the mouse in the viewer area. The focus stays in the left panel, up/down keys move to the next/previous file. But the bottom button bar is updated to contain the viewer's actions. This is an inconsistent buggy state. Pressing one of the function keys executes the function for the viewer, and reverts the button bar to its normal state. Quite chaotic.
I think clicking in the viewer area should make it the focused one, exactly as when you make it focused with the Tab key.
Change History
comment:2 Changed 10 years ago by andrew_b
- Owner set to andrew_b
- Status changed from new to accepted
- Branch state changed from no branch to on review
- Milestone changed from Future Releases to 4.8.13
Branch:3251_quickview_grab_focus
changeset:2ffe03ca424d45e502445f4c5a0fcaee590561c8
comment:3 Changed 10 years ago by slavazanko
- Votes for changeset set to slavazanko
- Branch state changed from on review to approved
comment:4 Changed 10 years ago by andrew_b
- Votes for changeset changed from slavazanko to committed-master
- Branch state changed from approved to merged
Merged to master: [f5c04331e10d8f62c9f9ad9b8540ce5702eb9646].
Interestingly, when in this inconsistent state, pressing one of the function keys reverts to the normal button bar (e.g. press F2, Wrap or Unwrap becomes Menu), but clicking with the mouse on these entries keep the viewer's button bar (e.g. you can keep clicking Wrap and Unwrap).