🐛 Fix logic for displaying custom alt text #121
Conversation
This is a rework for displaying the custom alt text for works in the dashboard.
|
Should this include the required changes for collection index? @kirkkwang |
|
@ShanaLMoore This does not, that was actually in the Hyku app itself. |
Oh ok. I think this PR should include a fix for that too. Will a partial override work? |
Hmm, there currently isn't a list_collections partial in AllinsonFlex, I thought that this gem didn't even affect Collections. |
There was a problem hiding this comment.
Sorry @kirkkwang I didn't realize I was reviewing an allinson flex PR 🙈 I've had a busy meeting filled morning and must not have been paying attention to the context. It totally makes sense now to not have collection work here.
This commit will fix the alt text for the collection thumbnail on the dashboard and the Allinson Flex update will fix the thumbnail for the work on the dashboard. Ref: - samvera-labs/allinson_flex#121 - #471
This is a rework for displaying the custom alt text for works in the dashboard. Also the fallback for the text is now aligned with Hyrax
https://github.com/samvera/hyrax/blob/cbdc3194b45d623e68467bca244df2831924d0a7/app/views/hyrax/my/works/_list_works.html.erb#L12