KEMBAR78
Embedding Projector: fix regex suffix css by alicialics · Pull Request #6329 · tensorflow/tensorboard · GitHub
Skip to content

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

@alicialics alicialics force-pushed the fix_input_regex_css branch from 426d8ef to 209a051 Compare April 16, 2023 01:43
@alicialics alicialics force-pushed the fix_input_regex_css branch from 209a051 to fa0993d Compare April 18, 2023 02:57
@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.

4 participants