[MGNLDEMO-297] Sample assets make no use of relevant metadata Created: 24/Aug/18 Updated: 17/May/19 Resolved: 25/Jan/19 |
|
| Status: | Closed |
| Project: | Magnolia Demo Projects |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.4 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Maxime Michel | Assignee: | Antti Hietala |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | 0.5d | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoR: |
Empty
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
As seen at the following URL's:
This has been made visible by We should use those properties correctly to make our demos more appealing and more accessible (think Braille readers). This overhaul is needed for all assets, from tours (CE) to stories (EE). |
| Comments |
| Comment by Christopher Zimmermann [ 19/Sep/18 ] |
|
I think the caption value is used in the travel demo templates. Example: search for "Brian" on this page: I'm not sure of the reason it was done that way in the past - I think there may have been some problems with accessing the DAM metadata from a template . |
| Comment by Maxime Michel [ 19/Sep/18 ] |
Shouldn't that be verified (and solved?) as a prerequisite to this ticket then? |
| Comment by Antti Hietala [ 19/Sep/18 ] |
|
Yes, caption is used in templates to render a credit to the photographer.
Let's say the new, more appropriate caption will read "Kleine Scheidegg hiking trails by Ed Coyle". In that case we may want to adjust the i18n label preceding it. Current: New: |
| Comment by Maxime Michel [ 19/Sep/18 ] |
|
Shouldn't "Kleine Scheidegg hiking trails by Ed Coyle" rather be a combination of the caption and author fields? |
| Comment by Antti Hietala [ 19/Sep/18 ] |
Yes, I think you are correct. Caption should rather describe what's in the photo, not who took it. Btw, Dublin Core metadata (DCMI) doesn't have a caption property at all. This means we a free to define caption in a way that fits the travel demo site best. On the other hand, we are limited by available space in the design. Here's another example of caption fallback in a Text & Image component:
I'll define a few meaningful captions and see if we can avoid having to change the templates. |
| Comment by Antti Hietala [ 18/Jan/19 ] |
|
Reopened. The following fixes should be done:
|