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

fix: show the name instead of the code in print view #1371

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

yanfali
Copy link
Collaborator

@yanfali yanfali commented Nov 7, 2024

Issue 1363

Change the PrintKey to show the .name instead of the .code attribute because it's less human friendly. Remove some of the extraneous formatting around the element.

This stems slightly from the different way we render Key components in the printing vs editing.

Description

Issue [1363](#1363)

Change the PrintKey to show the `.name` instead of the `.code` attribute
because it's less human friendly. Remove some of the extraneous
formatting around the element.

This stems slightly from the different way we render Key components in
the printing vs editing.
@yanfali
Copy link
Collaborator Author

yanfali commented Nov 7, 2024

Before

image

After

image

Copy link
Member

@noroadsleft noroadsleft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@yanfali yanfali merged commit 6add44d into master Nov 8, 2024
4 checks passed
@yanfali yanfali deleted the issue-1363-printing-container-keys branch November 8, 2024 08:53
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