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

Bottom panel loading z-index conflicts with main area context menu #250

Open
enchained opened this issue Jan 30, 2025 · 3 comments
Open
Labels
fix-committed Fix for the issue is in the main branch

Comments

@enchained
Copy link

One of my machines is pretty slow, especially when HDD is busy. I sometimes notice that if I right click any other torrent at the bottom of the list, its details in the bottom panel (General, Files, etc.) do not load right away. And while it shows the loading animation on top of it, the bottom panel overlaps the bottom of the context menu, making it unclickable.

It usually happens when I remove torrents one by one using the right click menu, and I have to wait a few seconds till I can click "Remove..." on the bottom of the menu.

My guess is that the loading div z-index is a bit too high, but the loading is usually gone before I can Inspect the styles in devtools.

qu1ck added a commit that referenced this issue Jan 31, 2025
@qu1ck qu1ck added the fix-committed Fix for the issue is in the main branch label Jan 31, 2025
@qu1ck
Copy link
Member

qu1ck commented Jan 31, 2025

Thanks, fixed.
Note: you can invoke the remove action from the toolbar, saves you a click.

@enchained
Copy link
Author

That was fast, thanks! Will test on release.

Note: you can invoke the remove action from the toolbar, saves you a click.

But this is true only when you already have item(s) pre-selected, right? But in my case I make a decision to remove, select a single item with the right click, and click Remove. Both ways cost 2 clicks, but I prefer mine cause I'm easily distracted in the middle of something, so controlling the context this way feels better. Plus there is less mouse travel overall.

@qu1ck
Copy link
Member

qu1ck commented Jan 31, 2025

But this is true only when you already have item(s) pre-selected, right?

Yes

Both ways cost 2 clicks ... Plus there is less mouse travel overall.

If you are deleting one by one then yeah, fair enough. If you want to reduce mouse travel even further and don't mind using keyboard then you can just click to select and press del + enter to confirm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix-committed Fix for the issue is in the main branch
Projects
None yet
Development

No branches or pull requests

2 participants