Ticket #3251 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

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:1 Changed 5 years ago by egmont

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).

comment:2 Changed 5 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 5 years ago by slavazanko

  • Votes for changeset set to slavazanko
  • Branch state changed from on review to approved

comment:4 Changed 5 years ago by andrew_b

  • Votes for changeset changed from slavazanko to committed-master
  • Branch state changed from approved to merged

comment:5 Changed 5 years ago by andrew_b

  • Status changed from accepted to testing
  • Resolution set to fixed

comment:6 Changed 5 years ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.