-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Match flag colors with Anki Desktop #18018
Comments
May I take this up? |
@BrayanDSO will you assign us the issue, or else we can directly raise a pr after completion? |
@BrianDSO I would like to work on this issue |
Hey @BrayanDSO Can i look into this issue?I would like to work on it. |
hey @BrayanDSO can i work on this? |
@devyk100 asked first, so they can go |
Are we definitely doing this rather than using the app bar? The 'mark' icon is ugly, and the flags feel better-placed in the current reviewer. [for both placements] Given the screenshots in the PR, I prefer our existing colors (and believe they were added specifically to move away from the Desktop color scheme): |
The icons will be the same used in AnkiDroid, no the ones used in Anki (Anki icons depends on the device fonts, so they are not safe to use anyway) About the placement, what do you propose? mini icons in the top-center (or bottom) of the screen? Adding a minibar would consume screen space and I'd like to avoid that if it's just for flags/marks. Their place was a painpoint when designing the screen, so I went with the easiest option. About the colors, I'm indifferent. Using AnkiDroid's colors is simple. I just went with the p.s.: idk where to put the timer. I'm just ignoring its existence for now |
Colors are a mild preference, and if you'd prefer consistency and it's a conscious choice, then I'll defer to you. The design of the screen is a series of great design decisions. Off-topic for this issue Flag placement: I'll sleep on this one. I suspect we need to bring the bar back, given the quantity of possible items which should be visible & non-intrusive (flag/mark/timer) For the flag in particular, we might be able to get away with something subtle (colored underlines/shadow changes), but that wouldn't fix the timer issue, so may not be worth looking into, unless it's an idea someone loves |
for tablets, which are commonly used in landscape mode, I don't want a horizontal minibar consuming space. The timer could be in the bottom bar since there's plenty of space there, and the flag mark could be in the corners like the desktop version. for phones, the minibar is probably the best idea. |
Both light and dark flags.
The task is to use the same colors for flags that Anki desktop uses.
The colors are going to be relevant when the new reviewer shows the flags with the JavaScript methods from upstream.
The text was updated successfully, but these errors were encountered: