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 regex suffix css #6329

Merged
merged 1 commit into from
Apr 19, 2023

Conversation

alicialics
Copy link
Contributor

@alicialics alicialics commented Apr 16, 2023

Motivation for features / changes

The regex end slash and the regex button did not fit in the same line.

Technical description of changes

There should be a single suffix slot element that flex in the correct direction

Screenshots of UI changes

Before

Screenshot 2023-04-17 at 8 18 31 PM

After

image

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

Open the regex option in input

Alternate designs / implementations considered

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

The regex end slash and the regex button did not fit in the same line.

## Technical description of changes

There should be a single suffix slot element that flex in the correct
direction

## Screenshots of UI changes

### Before
<img width="141" alt="Screenshot 2023-04-17 at 8 18 31 PM"
src="https://user-images.githubusercontent.com/31378877/232662251-e2b6dcf4-d6e4-4ea2-8334-cf79e7da26fe.png">

### After

![image](https://user-images.githubusercontent.com/31378877/232662384-e20d71d6-b152-4bcf-9f04-71bd5ee8b156.png)


## Detailed steps to verify changes work correctly (as executed by you)
Open the regex option in input

## 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

4 participants