-
Notifications
You must be signed in to change notification settings - Fork 0
fix: minor UI and styling tweaks #126
base: main
Are you sure you want to change the base?
Conversation
Add a BooksBreadcrumb function that renders PageBreadcrumb with Home and Books. It is a separate function because the breadcrumb is needed in every return of the component
The cards are smaller on the profile page, which made the rating text wrap awkwardly
Chrome painted a light blue autofill background on the search input when a previous search was picked in dark mode. With a color-scheme for light and dark mode the browser uses matching native styling. This does not remove the autofill tint completely, but it is much better in both themes.
Render a title in the selected book page component instead of exporting meta, since the title depends on the book data that is defined dynamically in the route. React 19 "hoists" the tag into the head, so the tab title updates with the book.
|
I also noticed our drop-downs currently don't use |
Good catch, we should add that as well |
Co-authored-by: Martin Alexander Kaminski <martakam@stud.ntnu.no>
|
I have removed the duplicate title from I also added a log out button to the profile page, since it felt like a logical feature to have there. |
|
Removing this fixed the duplicate titles 👍 but now nothing sets a title on the 404/error page: root has no Could you add a title in <title>{`${message === "404" ? "Page not found" : "Error"} | Readr`}</title>(A This comment was written with help of AI |
This PR adds a set of small UI, styling and accessibility fixes:
color-schemefor light and dark mode, which reduces the light blue autofill background on the search input in dark mode<title>in the component instead ofmeta, since the title depends on data about the bookCloses #125
Closes #96