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

refactor: more pill css changes #1183

Merged
merged 4 commits into from
Jun 6, 2024
Merged

refactor: more pill css changes #1183

merged 4 commits into from
Jun 6, 2024

Conversation

setchy
Copy link
Member

@setchy setchy commented Jun 5, 2024

Some further changes are sleeping on #1177

Changes

  • update tailwind css classes
  • shrink pills to xss (via custom tailwind font size)
  • allow pills to flex wrap to new line if too long
  • change updated datetime to use strict formatting (removes the about prefix and is more precise) - saves some horizontal space.
  • truncate long repo names or notification titles

Exaggerated example (i locally duplicated the linked issues pills to demonstrate)

Screenshot 2024-06-05 at 10 30 56 AM

@setchy setchy added the refactor Refactoring of existing feature label Jun 5, 2024
@setchy setchy added this to the Release 5.7.0 milestone Jun 5, 2024
@setchy
Copy link
Member Author

setchy commented Jun 5, 2024

In a separate PR I'll raise a new setting to allow users to toggle off pills all together

@setchy setchy marked this pull request as draft June 5, 2024 13:50
@setchy setchy marked this pull request as draft June 5, 2024 13:50
@setchy setchy marked this pull request as ready for review June 5, 2024 14:31
@setchy
Copy link
Member Author

setchy commented Jun 6, 2024

Will fix the merge conflict shortly then I think cut the release

@setchy setchy merged commit e2b0892 into main Jun 6, 2024
6 checks passed
@setchy setchy deleted the refactor/continued-ui-tweaks branch June 6, 2024 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Refactoring of existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants