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

Orthographic: Selection rectangle grabs all atoms #1129

Open
ghutchis opened this issue Aug 24, 2022 · 3 comments
Open

Orthographic: Selection rectangle grabs all atoms #1129

ghutchis opened this issue Aug 24, 2022 · 3 comments

Comments

@ghutchis
Copy link
Member

At least on a Mac, if I use the selection rectangle, all atoms seem to be selected.

I can confirm this behavior does not exist in 1.94, but does exist in 1.95 and afterwards.

I'm not sure the bug is in selectiontool.cpp - it might be in the call to m_renderer->hits() instead.

@ghutchis
Copy link
Member Author

ghutchis commented Aug 24, 2022

Aha, git bisect finds that it derives from #709 b30e202 (adding the layer system)

Weird bug - if I create multiple layers (e.g., select hydrogens, move them to Layer 2) it seems to work. (At least some/most of the time?)

Screen Shot 2022-08-24 at 2 19 21 PM

@ghutchis
Copy link
Member Author

ghutchis commented Aug 26, 2022

More importantly, this shows up with orthographic projection, not perspective.

So the problem likely comes from the screen projection.

@ghutchis ghutchis changed the title Selection rectangle grabs all atoms Orthographic: Selection rectangle grabs all atoms Apr 9, 2023
@ghutchis ghutchis added the bug label May 10, 2023
@avo-bot
Copy link

avo-bot commented Aug 26, 2024

This issue has been mentioned on Avogadro Discussion. There might be relevant details there:

https://discuss.avogadro.cc/t/selection-by-drag-rectangle-doesnt-work-in-orthographic-projection/5542/2

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

2 participants