Skip to content

chore: adjust the style and wording of the license page #7396

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

Merged
merged 2 commits into from
May 14, 2023
Merged

Conversation

kylecarbs
Copy link
Member

@kylecarbs kylecarbs commented May 3, 2023

  • Reduce the size of the card to improve the use of space
  • Adjust the language of the add page to remove word redundancy
  • Change from using background colors to text colors
  • The license removal didn't indicate to users what it would actually do, so I added a message.

Check storybook to see the visual diff!

image

@kylecarbs kylecarbs requested a review from rodrimaia May 3, 2023 19:55
@kylecarbs kylecarbs self-assigned this May 3, 2023
- Reduce the size of the card to improve the use of space
- Adjust the language of the add page to remove word redundancy
- Change from using background colors to text colors
Copy link
Contributor

@rodrimaia rodrimaia left a comment

Choose a reason for hiding this comment

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

Nice changes. I liked the copy changes. :)

(just for clarification, the idea of having a bigger card at the beginning was to have nice UI for one license only since most users will only have one, not a list of them)

lgtm!
Maybe the only thing is that in this configuration , the Remove button looks a little bit not aligned.
image

but changing it is up to you. Seems fine as it is :)

@github-actions
Copy link

This Pull Request is becoming stale. In order to minimize WIP, prevent merge conflicts and keep the tracker readable, I'm going close to this PR in 3 days if there isn't more activity.

@github-actions github-actions bot added the stale This issue is like stale bread. label May 12, 2023
@kylecarbs kylecarbs removed the stale This issue is like stale bread. label May 12, 2023
@kylecarbs kylecarbs merged commit 778cb74 into main May 14, 2023
@kylecarbs kylecarbs deleted the licensestyle branch May 14, 2023 20:31
@github-actions github-actions bot locked and limited conversation to collaborators May 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants