feat: Segment List View
Author: @jstarpl
-
What kind of change does this PR introduce? GUI feature
-
What is the current behavior? There are currently two views in the Rundown: Timeline-view and Storyboard view (Card-view.)
-
What is the new behavior? This PR adds a third view in the Rundown-view GUI; the "List view". This view displays each Part in a Segment on it's own line vertically, instead of the Segment being a line and Parts populated horizontally. This view is useful for shows that have Parts that vary a lot in length, such as sports shows.
-
Other information:
The view is hidden behind a URL-parameter for now, to allow for easy user-testing. It's enable by adding ?useOnePartPerLine=1 to the end of the URL.
There are a few edge-cases not implemented yet, so there will be a few patches here and there in future releases, but it is good-enough by now to warrant a merge.
Status
- [ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
- [ ] The functionality has been tested by the PR author
- [ ] The functionality has been tested by NRK
Codecov Report
Base: 68.16% // Head: 68.15% // Decreases project coverage by -0.00% :warning:
Coverage data is based on head (
4c23022) compared to base (886d9cc). Patch coverage: 20.00% of modified lines in pull request are covered.
Additional details and impacted files
@@ Coverage Diff @@
## release45 #753 +/- ##
=============================================
- Coverage 68.16% 68.15% -0.01%
=============================================
Files 277 277
Lines 33729 33734 +5
Branches 4499 4500 +1
=============================================
+ Hits 22991 22992 +1
- Misses 10285 10289 +4
Partials 453 453
| Impacted Files | Coverage Δ | |
|---|---|---|
| meteor/client/lib/localStorage.ts | 27.17% <20.00%> (-0.42%) |
:arrow_down: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.