Skip to content

Reduce Regenerate Token UI density #15104

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

Closed
Tracked by #14280
easyCZ opened this issue Dec 1, 2022 · 2 comments · Fixed by #15107
Closed
Tracked by #14280

Reduce Regenerate Token UI density #15104

easyCZ opened this issue Dec 1, 2022 · 2 comments · Fixed by #15107
Assignees

Comments

@easyCZ
Copy link
Member

easyCZ commented Dec 1, 2022

Screenshot 2022-12-01 at 14 13 00

The Rengerate Token UI looks a bit dense, especially around the volume of dates being shown. Cleaning up and breaking it up a bit could make it easier to visually grep.

CC @gtsiolis if you've got ideas what we could improve here. Also feel free to close if this is the intended UX

@gtsiolis
Copy link
Contributor

gtsiolis commented Dec 1, 2022

That's the most UX nerdy issue title I've read in a while. Thanks for opening this, @easyCZ!

Yeah, simply using our spacing guidelines for separating form groups using mt-4 could help.

For future reference, our spacing guidelines follow Tailwind's spacing scale. I'm posting below a screenshot from Foundations / Spacing in the component library in our Figma team.

Spacing Guidelines
Spacing

Posting a diff below, although I'm not sure if the code is correct:

diff --git a/components/dashboard/src/settings/PersonalAccessTokens.tsx b/components/dashboard/src/settings/PersonalAccessTokens.tsx
index ea391388d..97243dcb9 100644
--- a/components/dashboard/src/settings/PersonalAccessTokens.tsx
+++ b/components/dashboard/src/settings/PersonalAccessTokens.tsx
@@ -148,7 +148,7 @@ function ShowTokenModal(props: TokenModalProps) {
                     Expires on {dayjs(props.token.expirationTime!.toDate()).format("MMM D, YYYY")}
                 </div>
             </div>
-            <>
+            <div className="mt-4">
                 {props.showDateSelector && (
                     <DateSelector
                         title="Expiration Date"
@@ -165,7 +165,7 @@ function ShowTokenModal(props: TokenModalProps) {
                         }
                     />
                 )}
-            </>
+            </div>
         </Modal>
     );
 }

Here's how the design should be after this change:

BEFORE AFTER
modal-before modal-after

@gtsiolis
Copy link
Contributor

gtsiolis commented Dec 1, 2022

Opened #15107 to resolve this.

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 a pull request may close this issue.

2 participants