Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add sourcelayer "studio screen" (SOFIE-3132) #1189

Open
wants to merge 4 commits into
base: release51
Choose a base branch
from

Conversation

nytamin
Copy link
Contributor

@nytamin nytamin commented May 16, 2024

About the Contributor

This pull request is posted on behalf of the NRK.

Type of Contribution

This is a:

Feature

Current Behavior

Graphics that are intended for backscreens in the studio have been put on the GRAPHICS layer (which is used for fullscreen gfx). However there is a need to distinguish the back screen graphcs from lower-thirds and fullscreen gfx.

New Behavior

A new SourceLayer "STUDIO_SCREEN" has been added, intended to be used for content targeting studio screens.

Testing

  • I have added one or more unit tests for this PR
  • I have updated the relevant unit tests
  • No unit test changes are needed for this PR

Affected areas

This PR affects the GUI / display of Pieces.

Time Frame

No rush, but we're aiming at merging this into R51 within a week.

Other Information

Status

  • PR is ready to be reviewed.
  • The functionality has been tested by the author.
  • Relevant unit tests has been added / updated.
  • Relevant documentation (code comments, system documentation) has been added / updated.

@nytamin nytamin requested a review from a team as a code owner May 16, 2024 06:09
@nytamin nytamin marked this pull request as draft May 16, 2024 06:09
@Julusian Julusian changed the base branch from release51 to release50 May 16, 2024 11:37
@Julusian Julusian changed the base branch from release50 to release51 May 16, 2024 11:38
@nytamin nytamin marked this pull request as ready for review May 17, 2024 09:13
Copy link

codecov bot commented May 21, 2024

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 58.10%. Comparing base (d1a593c) to head (f51bbbf).
Report is 387 commits behind head on release51.

Files Patch % Lines
meteor/server/api/rest/v1/typeConversion.ts 0.00% 6 Missing ⚠️
...artThumbnail/Renderers/ThumbnailRendererFactory.ts 0.00% 1 Missing ⚠️
...r/client/ui/Shelf/Renderers/ItemRendererFactory.ts 0.00% 1 Missing ⚠️
meteor/lib/api/rest.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##           release51    #1189      +/-   ##
=============================================
- Coverage      58.10%   58.10%   -0.01%     
=============================================
  Files            483      483              
  Lines          79442    79451       +9     
  Branches        4000     3548     -452     
=============================================
+ Hits           46162    46166       +4     
- Misses         33255    33280      +25     
+ Partials          25        5      -20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@Julusian Julusian left a comment

Choose a reason for hiding this comment

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

Should the new type be included in any of the following locations too?

I'm honestly not sure if it should, I don't know what the PieceIcons stuff is for or where they are used. The PieceMultistepChevron one will be wanted though.

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.

None yet

2 participants