Skip to content

feat: legend component for standings table#7477

Open
ElectricalBoy wants to merge 23 commits intomainfrom
legend-widget
Open

feat: legend component for standings table#7477
ElectricalBoy wants to merge 23 commits intomainfrom
legend-widget

Conversation

@ElectricalBoy
Copy link
Copy Markdown
Collaborator

Summary

This PR implements the legend component for standings tables.

How did you test this change?

sandbox + browser dev tools

@ElectricalBoy ElectricalBoy requested review from a team May 8, 2026 05:18
@ElectricalBoy ElectricalBoy added c: standings stylesheets Changes to stylesheets labels May 8, 2026
@ElectricalBoy ElectricalBoy changed the title feat: legend component for standings feat: legend component for standings table May 8, 2026
Copy link
Copy Markdown
Collaborator

@hjpalpha hjpalpha left a comment

Choose a reason for hiding this comment

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

do we want/need to adress the dq case?


GoldenTest('standings_legend', tostring(LegendComponent{
color = {
byeup = 'Lorem ipsum',
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

maybe to make sure no one adds a bad mapping

Suggested change
byeup = 'Lorem ipsum',
byeup = 'Lorem ipsum byeup',

(analogous for others)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: standings stylesheets Changes to stylesheets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants