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

Better align book details when shown in a grid #166

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

csmith
Copy link
Contributor

@csmith csmith commented Dec 23, 2024

In a book grid, the title, author and status are shown in a column with no real layout constraints. This means if a book has a long title, its author and status won't line up with a different book with a short title, which makes scanning across the grid annoying.

To fix this, give the title the flex-grow property so it'll expand to fill the available room. Authors and status will then be bottom-aligned.

Before After

In a book grid, the title, author and status are shown in a column
with no real layout constraints. This means if a book has a long
title, its author and status won't line up with a different book
with a short title, which makes scanning across the grid annoying.

To fix this, give the title the flex-grow property so it'll expand
to fill the available room. Authors and status will then be
bottom-aligned.
@bayang bayang merged commit c67f955 into bayang:main Dec 23, 2024
@bayang
Copy link
Owner

bayang commented Dec 23, 2024

It's nicer indeed. Thanks !

github-actions bot pushed a commit that referenced this pull request Dec 23, 2024
### [0.61.3](v0.61.2...v0.61.3) (2024-12-23)

### Bug Fixes

* Better align book details when shown in a grid ([#166](#166)) ([c67f955](c67f955))
@bayang
Copy link
Owner

bayang commented Dec 23, 2024

🎉 This PR is included in version 0.61.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants