Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Unselect hovered" is broken #179

Closed
whitequark opened this issue Feb 1, 2017 · 11 comments
Closed

"Unselect hovered" is broken #179

whitequark opened this issue Feb 1, 2017 · 11 comments

Comments

@whitequark
Copy link
Contributor

It just never appears anymore.

@whitequark whitequark added the bug label Feb 1, 2017
@whitequark whitequark added this to the 3.0 milestone Feb 1, 2017
@Evil-Spirit
Copy link
Collaborator

@whitequark, this works for me

@whitequark
Copy link
Contributor Author

@Evil-Spirit Try this:

  1. New file
  2. Draw a rect
  3. Click on top line
  4. Hover over top line and right-click

@whitequark
Copy link
Contributor Author

@Evil-Spirit Oh... I've just realized "Unselect Hovered" doesn't appear unless more than one entity is selected. Nevermind.

cc @wpwrak

@wpwrak
Copy link
Contributor

wpwrak commented Feb 1, 2017

Hmm, but even if selecting two lines of the rect, I don't see any "Unselect Hovered", just "Unselect All":
no-unselect-hovered

@whitequark
Copy link
Contributor Author

@wpwrak Are you really hovering the top line? The menu seems quite far from it.

@wpwrak
Copy link
Contributor

wpwrak commented Feb 2, 2017

Yes, the line pales with fear as the mouse pointer approaches. Alas, it turns regular red again when right-clicking, so one can't see that in the screenshot.

By the way, for this type of debugging, it may be useful to have an option somewhere (could be on the command line), to make SolveSpace draw a cursor (maybe X-shaped), so that it is included in the screenshot.

@whitequark
Copy link
Contributor Author

By the way, for this type of debugging, it may be useful to have an option somewhere (could be on the command line), to make SolveSpace draw a cursor (maybe X-shaped), so that it is included in the screenshot.

This is really the responsibility of your screenshot program. Both GNOME and KDE provide such an option, for one.

@wpwrak
Copy link
Contributor

wpwrak commented Feb 2, 2017

Sigh, all that modernism (using xwd here) ... here we have one with cursor, courtesy of "screengrab":
no-unselect-hovered-cursor

@whitequark
Copy link
Contributor Author

@wpwrak can you please use gdb to trace GraphicsWidget::{on_motion_notify_event,on_button_press_event,on_button_release_event} calls?

@whitequark whitequark added UI and removed UI labels Jul 12, 2018
@whitequark
Copy link
Contributor Author

@wpwrak can you still reproduce this?

@whitequark whitequark removed this from the 3.0 milestone May 23, 2019
@whitequark
Copy link
Contributor Author

No response in two years, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants