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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

AndroidEnv Example Tasks doc: fix download UI for APK/textproto ZIP files in dark mode #5

Merged
merged 2 commits into from May 31, 2021

Conversation

8bitmp3
Copy link
Contributor

@8bitmp3 8bitmp3 commented May 29, 2021

The download buttons for APK/textproto ZIP files are barely visible in GitHub web UI's dark mode, so they may be hard to find:

image

A simple solution could be just to have the word "Download" - it renders well in Markdown:

image

Hope this helps 馃憤

@kenjitoyama
Copy link
Collaborator

@a-gergely , maybe we could use another icon that works well with a dark theme, but for now I think @8bitmp3's solution is more accessible, so I'll accept and merge this PR.

@kenjitoyama kenjitoyama merged commit 5138bd6 into google-deepmind:main May 31, 2021
@8bitmp3
Copy link
Contributor Author

8bitmp3 commented May 31, 2021

@kenjitoyama @a-gergely Thank you. I've looked at other icons in the Material design library but couldn't find anything suitable for GitHub's dark mode UI.

@8bitmp3 8bitmp3 deleted the patch-2 branch May 31, 2021 15:05
@kenjitoyama
Copy link
Collaborator

@8bitmp3, yeah, I also took a quick look and couldn't find anything.
I'm not sure about GitHub's Markdown implementation, but if we could insert a CSS transform, we could flip the colors, but I agree that just putting the word "Download" is better for now. Thanks!

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.

None yet

2 participants