[MGNLDEMO-227] Stories App demo can be broken with content Created: 30/Aug/17  Updated: 26/Jan/18  Resolved: 26/Jan/18

Status: Closed
Project: Magnolia Demo Projects
Component/s: None
Affects Version/s: 1.2.1
Fix Version/s: 1.1.7, 1.2.2

Type: Bug Priority: Neutral
Reporter: Christopher Zimmermann Assignee: Antonín Juran
Resolution: Fixed Votes: 0
Labels: None
Σ Remaining Estimate: Not Specified Remaining Estimate: Not Specified
Σ Time Spent: Not Specified Time Spent: Not Specified
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Attachments: PNG File 01_stories_overview_with_9_stories.png    
Sub-Tasks:
Key
Summary
Type
Status
Assignee
MGNLDEMO-255 Exception is thrown if lead has less ... Sub-task Closed Antonín Juran  
MGNLDEMO-256 Handle image, video, or embed for the... Sub-task Closed Antonín Juran  
MGNLDEMO-257 Page should list all of the other sto... Sub-task Closed Antonín Juran  
MGNLDEMO-259 Exception is thrown if there is no te... Sub-task Closed Evzen Fochr  
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Epic Link: Fix stories demo issues
Sprint: Kromeriz 126, Kromeriz 127, Kromeriz 128, Kromeriz 129, Kromeriz 130, Kromeriz 131
Story Points: 13

 Description   

The stories app demo templates work well with the provided sample content, but has some issues if a user enters content in the Stories app that templates do not anticipate. The templates should be robust enough to handle any user created story without generating exceptions, and displaying well.

Known issues

Story overview page

  • Exception If lead has less than 100 chars.
  • Should be able to handle image, video, or embed for the lead visual. Today it only works if Embed > Alternative image is specified.
  • Stories and Discover text may be too big to fit on mobile screen MGNLDEMO-228
  • Page should list all of the other stories that are not selected in the lead 3 items.

Story page

  • Crashes if story is not in “stories-demo” directory. The Stories choose dialog should not allow the user to choose a story from outside the stories-demo directory.
  • Exception if there is no text block in the story. The first text block is used for the Clock Intro. If none is found, leave the intro blank, don't throw an exception.
  • Exception if there is no embedsource. Story page should be flexible enough to handle image, video asset, or embed for the "lead visual". Same as with overview page above.


 Comments   
Comment by Antti Hietala [ 26/Oct/17 ]

avongunten, could you provide a design for "Page should list all of the other stories that are not selected in the lead 3 items.", please? If I have four stories, how should the fourth story be displayed on the Stories overview page?

Comment by Anja von Gunten [ 17/Nov/17 ]

I'm suggesting to repeat the pattern of the first 3 stories and alternately switch the stories teasers (see mockup). So the first block has the big teaser on the left side, second block has the big teaser on the right side, etc.

Note: The order of blocks is just a suggestion, not mandatory to follow. If the grid on the page doesn't allow the pattern

4 5
6

then use

4 6
5
Generated at Mon Feb 12 05:17:35 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.