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

Fix deck button not clickable in stats screen for smaller screens #3602

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

Sawansunar56
Copy link
Contributor

@Sawansunar56 Sawansunar56 commented Nov 29, 2024

fix #3595

Changed the z-index of the spinner element in the stats screen, which caused the spinner to take the click events if the spinner and the radio buttons(deck, collection) were on top of each other, due to insufficient space. I just made it so that it goes behind if the element is not loading and in front if it is.

Changed z-index of the underlying spinner
@mikehardy
Copy link
Contributor

@Sawansunar56 can you add a quick screen capture or image showing the UI affects of this, and explain in your PR description above (with an edit...) exactly what was done and why? That would help review a lot

Thanks!

@Sawansunar56
Copy link
Contributor Author

Sawansunar56 commented Nov 29, 2024

anki_capture.mp4

NOTE: I increased the opacity of the spinner for this instance to show the change.

Copy link
Contributor

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice - thanks for the description and the video, makes it clear

@Sawansunar56
Copy link
Contributor Author

Nice - thanks for the description and the video, makes it clear

I appreciate the input as well.

Copy link
Collaborator

@abdnh abdnh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good - Thank you @Sawansunar56 !

@abdnh abdnh merged commit f6a3e98 into ankitects:main Dec 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't Select Deck in Stats Screen (Mobile)
3 participants