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

New dataplane tag UI #10

Merged
merged 4 commits into from
Feb 4, 2020
Merged

New dataplane tag UI #10

merged 4 commits into from
Feb 4, 2020

Conversation

bloqhead
Copy link
Contributor

@bloqhead bloqhead commented Jan 29, 2020

This change gives the tags on a dataplane a more user-friendly experience and will include color-coding based on the tag type (env, service, version, etc). The revision also makes sure to account for dataplanes that are in gateway mode.

…tag and its value. this change accounts for gateway mode dataplanes as well.
@bloqhead bloqhead requested a review from a team January 29, 2020 22:00
@subnetmarco
Copy link
Contributor

Please add a border and the background of the box on the right (where the value is located) becomes white. Also remove the semicolon from the tag name. Similar to:

Screen Shot 2020-01-30 at 7 25 46 PM

Daryn St. Pierre added 2 commits February 3, 2020 10:54
…ble use in the tags list on data tables. style revisions for the tags.
…nt them from stacking too close to one another when there are multiple tags on smaller screens.
@bloqhead
Copy link
Contributor Author

bloqhead commented Feb 3, 2020

@subnetmarco I've revised the tag styles to match your example. I also increased spacing for when there are multiple tags so that they don't stack awkwardly. Let me know what you think.

Screenshot

@bloqhead bloqhead self-assigned this Feb 3, 2020
@subnetmarco
Copy link
Contributor

This works.

@bloqhead bloqhead merged commit 84d9d4b into master Feb 4, 2020
@bloqhead bloqhead deleted the feat/data-table-tag-ui branch May 4, 2020 18:39
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.

2 participants