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

Embedding Projector: fix dark mode button contrast #6327

Merged
merged 1 commit into from
Apr 19, 2023

Conversation

alicialics
Copy link
Contributor

@alicialics alicialics commented Apr 15, 2023

Motivation for features / changes

Help button cannot be seen in dark mode

Technical description of changes

force white background so the button is visible

Screenshots of UI changes

Before:
image

After:
Screenshot 2023-04-17 at 8 29 01 PM

Detailed steps to verify changes work correctly (as executed by you)

turn on dark mode and verify the dialog button is visible

Alternate designs / implementations considered

@rileyajones rileyajones merged commit c545a2c into tensorflow:master Apr 19, 2023
14 checks passed
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
## Motivation for features / changes

Help button cannot be seen in dark mode

## Technical description of changes

force white background so the button is visible

## Screenshots of UI changes

Before:

![image](https://user-images.githubusercontent.com/31378877/232663149-f20a6ae5-da41-4ae7-bd57-f51aec914efc.png)

After:
<img width="208" alt="Screenshot 2023-04-17 at 8 29 01 PM"
src="https://user-images.githubusercontent.com/31378877/232663682-c67c5326-5713-4e6e-9ae5-3ac03dfc80b6.png">


## Detailed steps to verify changes work correctly (as executed by you)
turn on dark mode and verify the dialog button is visible

## Alternate designs / implementations considered
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants