pressbooks / pressbooks-aldine

Aldine is the default root theme for Pressbooks.
https://pressbooks.org/
GNU General Public License v3.0
11 stars 7 forks source link

Improve the appearance of the default front page catalog #365

Closed SteelWagstaff closed 1 year ago

SteelWagstaff commented 1 year ago

As a network manager, I want the front page catalog block to better reflect the improved/attractive new catalog. Currently, if I turn on the front page catalog: Screenshot from 2022-10-26 12-21-33

the home page displays 3 colored blocks with book titles and about this book links and a carousel to display more books (scroll left/right buttons) with a link beneath to take the user to the 'complete catalog': Screenshot from 2022-10-26 12-23-18

We should make space for 4 featured books, should replace the color blocks with book covers and display the titles just beneath (as links to the book homepages). We can remove the carousel feature. The catalog link should point to whichever page uses the catalog template on their network. We need to handle cases where a network has multiple catalog pages (perhaps they can choose from a dropdown in the customizer for where the link goes?)

Network managers should be able to designate up to four books from their catalog as featured books in the customizer using a 4 drop down selectors that let them choose from any book in the network catalog as a featured book: 'Featured book 1' 'Featured book 2', etc.

richard015ar commented 1 year ago

An acceptable solution for multiple catalogues is to allow networks to have only one page designated as a catalog template page.

fdalcin commented 1 year ago

Reviewed and left a comment related to a minor issue I've detected while reviewing

SteelWagstaff commented 1 year ago

I don't see any of the book covers after choosing 4 books as 'featured' on integrations: Screenshot from 2022-11-08 13-45-52

See https://integrations.pressbooks.network/

arzola commented 1 year ago

I'll review it :whatshappeningJJ:

SteelWagstaff commented 1 year ago

Images are now appearing (I had to clear my browser cache). I would like us to make a couple of small adjustments to default size of the cover blocks.

A few suggestions to make:

Rationale: Our default book cover is 800px (tall) x 600px (wide), with a height to width ratio of 4:3 (or 1.33). Our current block has a default size of 318.91px x 286px (1.15). This is too small, especially because many books have cover ratios that fall between 1.4 - 1.6. There's probably a better, more scientific way to do this, but I think I want the default ratio for the cover image div to be somewhere in the range of 1.33 - 1.5.

arzola commented 1 year ago

What do you think to set by default an aspect ratio of 3/4 instead of doing rem, In my opinion looks good

https://github.com/pressbooks/pressbooks-aldine/pull/373

SteelWagstaff commented 1 year ago

Works as expected. Aspect ratio is now 4:3 with images centered so that excess content is cut off on top/bottom or left/right of overflow. We should consider improving instructions for cover images, if we expect/require them to have these 4:3 dimensions.