Skip to content

Conversation

@joemull
Copy link
Member

@joemull joemull commented Sep 10, 2025

Fixes #4952.

I've decided on a minimal intervention here, which is to make the author list scrollable when it is taller than 6 rem.

After a discussion we have decided to move the full author details to the sidebar for all three themes. It was already there for Material, so this is a change for Clean and OLH.

Screenshot from 2025-09-15 15-49-57 Screenshot from 2025-09-15 15-50-13 Screenshot from 2025-09-15 15-49-41

A compact list of authors still appears in the hero / title box.

I've moved the links away from the text and toward icons, to avoid distracting too much. I tried to make them accessible using the new patterns established by Steph. To make this play nicely your new list styling in #4905, @StephDriver, I have pulled foward some of your CSS and named you as co-author on that commit. I moved it to common, since it is identical as far as I could tell, where you had it in the 3 themes. I also modified it slightly to add an option for "and", and to wipe out unwanted artefacts from the theme styling.

I have also moved CRediT roles into the same display area, since it did not make sense to me to list authors again for this information. They have their own icon with a link out to the NISO CRediT site.

@joemull joemull requested review from ajrbyers and mauromsl and removed request for ajrbyers September 10, 2025 10:30
@joemull joemull added this to the v1.8.0 Tracker milestone Sep 10, 2025
Copy link
Member

@mauromsl mauromsl left a comment

Choose a reason for hiding this comment

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

I think displaying the email addresses is making the problem worse than it needs to be. some emails are incredibly verbose and I think using an icon/buttin is fine, plus the email can still be copied. I think the usability factor is how easy it is to read the person name and affiliation, rather than being able to read the email address. It also makes things less busy when using the sidebar
image
(note how this author has 3 URLS underlying, making it difficult to know what each does, unless you hover and inspect the URL below)
vs

image

We could also use a similar icon to link out to public profile, similar to how we do for ORCID.

As for the scroll bar on the OLH theme, I personally hate it but I'm unable to provide a better solution, other than moving the author metadata to the sidebar as suggested OR using an expansible list.

@mauromsl mauromsl assigned ajrbyers and unassigned mauromsl Sep 10, 2025
@joemull joemull assigned joemull and unassigned ajrbyers Sep 11, 2025
@joemull joemull force-pushed the b-4952-display-of-numerous-authors branch from 80a20e0 to 4db8479 Compare September 15, 2025 14:51
@joemull joemull requested a review from mauromsl September 15, 2025 14:58
@joemull joemull assigned mauromsl and unassigned joemull Sep 15, 2025
@joemull
Copy link
Member Author

joemull commented Sep 15, 2025

Thanks @mauromsl --please read the updated description above for an explanation of updates.

Copy link
Member

@mauromsl mauromsl left a comment

Choose a reason for hiding this comment

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

Thanks @mauromsl --please read the updated description above for an explanation of updates.

Lovely stuff, improved useability, more clarity, consistency across themes... 🥇 :

@mauromsl mauromsl assigned ajrbyers and unassigned mauromsl Sep 16, 2025
@mauromsl mauromsl requested a review from ajrbyers September 16, 2025 08:44
@ajrbyers ajrbyers merged commit b977614 into r-v1.8.x Sep 16, 2025
1 check passed
@ajrbyers ajrbyers deleted the b-4952-display-of-numerous-authors branch September 16, 2025 08:46
@joemull joemull linked an issue Sep 16, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OLH theme display bug when many authors have long affiliations

4 participants