Ticket #4013 (closed defect: fixed)

Opened 13 days ago

Last modified 9 days ago

Menu bar mouse clicks slightly off

Reported by: egmont Owned by: andrew_b
Priority: minor Milestone: 4.8.24
Component: mc-core Version: 4.8.23
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description (last modified by andrew_b) (diff)

In the menu bar, click on the space character immediately before the "File" label.

Actual: The "Left" menu opens.

Expected: The "File" menu should open, since when clicking on "File", this space character on its left is also highlighted.

Or, taking into account the dropdown menu's location (including its border), maybe even the second space character to the left could open this menu.

Same goes for all other top row entries of course, and also for mcedit.

Attachments

mc-4013-menu-bar-click.patch (510 bytes) - added by egmont 12 days ago.
Off by one fix

Change History

comment:1 Changed 12 days ago by andrew_b

  • Description modified (diff)
  • Summary changed from Top bar mouse clicks slightly off to Menu bar mouse clicks slightly off

Changed 12 days ago by egmont

Off by one fix

comment:2 Changed 12 days ago by egmont

The attached patch modifies the boundary by one.

If we want to offset it by yet another one, it would probably be nicer to do that in the assignment of start_x, affecting the entire rendering of the menu.

comment:3 Changed 9 days ago by andrew_b

  • Owner set to andrew_b
  • Status changed from new to accepted
  • Milestone changed from Future Releases to 4.8.24

comment:4 Changed 9 days ago by andrew_b

  • Branch state changed from no branch to on review

Branch: 4013_menubar_mouse_click
changeset:6a5d4243ce7b5fe1a41ad12a5420f66e87413b72

comment:5 Changed 9 days ago by andrew_b

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

comment:6 Changed 9 days ago by andrew_b

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b to committed-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

comment:7 Changed 9 days ago by andrew_b

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