-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[BUG]: There's no direct button to remove the background image for deck screen #16090
Comments
Hello! 👋 Thanks for logging this issue. Please remember we are all volunteers here, so some patience may be required before we can get to the issue. Also remember that the fastest way to get resolution on an issue is to propose a change directly, https://github.com/ankidroid/Anki-Android/wiki/Contributing |
I agree. The steps seem to be unnatural. |
I'll look into it. and get back to you. |
Yes! It feels strange.we can add button button just below the select image to remove background. |
Please assign me this issue. |
Hi @jainishrpandya I am currently working on this myself. |
@oyeraghib ohh! That's great. Ping me here if you need any help. |
Sure. Thanks |
While I agree that the current steps are unnatural, I don't necessarily agree with the suggested solution (#16091). The remove button being there also looks somewhat unnatural, in my opinion. Moreover, the option title itself also seems unnatural, since it is dissimilar to the other option titles. I mean, "Select image" expresses just an step in the middle of the change of this setting. It is not a final result of a change (e.g., "Show top bar", "Show deck title") nor a target of a change (e.g., "Night theme", "App bar buttons"). So, it actually looks odd as an option title. How about using another title like |
Yes! Toggle button with title like "Apply Image background" and when toggle is on it expands the select image option. Otherwise it will be hidden. |
Interesting point. So what you are suggesting is:
It is fine but the issue here would be if user wants to replace the image, they would have to first "OFF" the current toggle and then select a new image (turning it back ON), which could again become a bit unnatural from user's perspective. But I would still keep the suggestion in mind and would wait for any reviewers to comment on the PR and know what do they think. |
Thank you for the consideration. These parts below seem to be somewhat different from what I have imagined.
My thoughts are as follows: 4'. Now tapping the option title (tentatively named 5 . Instead, tapping the toggle button would remove the image (after a confirmation dialog for the removal), and turn it "OFF". I mean, when the toggle button is already turned "ON", the reactions of (A) tapping the title and (B) tapping the toggle button would be different from each other, as in the case of "Custom sync server" option (Settings > Sync > Custom sync server): Record_2024-04-05-07-03-20_a9eef3a2a561b80d5c76daebd0f9a14c.mp4 |
Hello 👋, this issue has been opened for more than 3 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically |
@david-allison Do we want this closed? |
We've spent too much time on a minor feature and the UX still isn't good. Let's focus on things which have more impact |
Checked for duplicates?
Does it also happen in the desktop version?
What are the steps to reproduce this bug?
3.Now to remove the background image you have to choose the "Select image" option again and close the selector in order to get the option to remove the background image.
Expected behaviour
Ideally there should be a button to remove the background image without actually selecting the "Select image" option
Actual behaviour
User have to tap on "Select Image" to remove the image (which sounds a bit off)
Debug info
(Optional) Anything else you want to share?
No response
Research
The text was updated successfully, but these errors were encountered: