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

Move "context menu" to BDV menu #1178

Open
tischi opened this issue Jul 12, 2024 · 3 comments
Open

Move "context menu" to BDV menu #1178

tischi opened this issue Jul 12, 2024 · 3 comments
Labels
discussion enhancement New feature or request

Comments

@tischi
Copy link
Contributor

tischi commented Jul 12, 2024

Hi @constantinpape, @imagejan, @tibuch,

I don't really understand (anymore) why we have this right click context menu:

image

I think now that it would be more intuitive (and more extensible) if this functionality would be available in the BDV menu bar, where there currently is File, Settings, Tools, Help.

I just checked and we can also remove some of them, e.g. Settings is useless for us.

So we could just add a MoBIE item there and put all the context menu stuff there.

My point is that "the right-click context" is in fact the whole BDV, thus having in the menu bar would be intuitive.


Ah.... I think I remember now, I think we initially had some functionality that depended where the user right-clicked and it would execute some action only for the image that was at the right-click location. But we don't do this anymore; I think we found this too complex, because (a) we would need to teach the users that this is the case and (b) there could be multiple sources at the same position....


Opinions?

@tischi tischi added enhancement New feature or request discussion labels Jul 12, 2024
@tibuch
Copy link
Collaborator

tibuch commented Jul 26, 2024

We use the Sow Raw Images and Log Current Location from time to time and it feels "natural" via the right click menu. But I agree that this could most likely be moved to a menu somewhere else. So, no strong opinion except for keeping the functionality in general ;)

@constantinpape
Copy link
Contributor

I also like the right click menu. Is there a strong reason to move it somewhere else?

@tischi
Copy link
Contributor Author

tischi commented Jul 26, 2024

No super strong reason. Mainly I could get rid of some dependencies that I currently need for the right-click and it would make the code a bit easier to maintain for me...

Maybe let's leave it as right click for now but also leave the issue open in case new arguments appear...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants