Skip to content

replace white#f9f9f9 for luma key alpha#936

Open
yetzt wants to merge 2 commits intorollerderby:devfrom
yetzt:dev
Open

replace white#f9f9f9 for luma key alpha#936
yetzt wants to merge 2 commits intorollerderby:devfrom
yetzt:dev

Conversation

@yetzt
Copy link
Copy Markdown
Contributor

@yetzt yetzt commented Apr 6, 2026

i'm using a hardware video switcher. in order to be able to use luma key instead of chroma key to remove the background, the purely white elements need to be slightly less white. otherwise all white elements will be interpreted as transparent and potentially illegible on stream. this is a great improvement in comparison with using chroma key, since it allows teams that use colors containing green. the visual difference is negligable.

Comment thread html/views/overlay/index.css Outdated
html { height: 100%; width: 100%; }
body { font-weight: bold; margin: 0; padding: 0; overflow: hidden; }
div { white-space: nowrap; display: block; padding: 0; margin: 0; overflow: hidden; }
div { #f9f9f9-space: nowrap; display: block; padding: 0; margin: 0; overflow: hidden; }
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please revert this to 'white' as 'white-space' is not associated with the colour white

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

oops, thanks for spotting this.

@yetzt yetzt requested a review from queekusme April 9, 2026 15:57
@yetzt
Copy link
Copy Markdown
Contributor Author

yetzt commented Apr 22, 2026

can we get this merged? i would really like to be able to have a green team on stream.

Copy link
Copy Markdown
Contributor

@queekusme queekusme left a comment

Choose a reason for hiding this comment

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

Whilst I haven't had time to lest this locally, I can't see anymore obvious concerns

@frank-weinberg
Copy link
Copy Markdown
Contributor

My plan is to merge this along with the other changes for 2027 when I am ready to create the first Beta.

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.

3 participants