Skip to content

Add custom scorecards generator - #122

Merged
jonatanklosko merged 3 commits into
jonatanklosko:masterfrom
maxidragon:custom-scorecards
Jul 8, 2026
Merged

Add custom scorecards generator#122
jonatanklosko merged 3 commits into
jonatanklosko:masterfrom
maxidragon:custom-scorecards

Conversation

@maxidragon

Copy link
Copy Markdown
Contributor

This PR adds a new tab to the "printing" section that allows users to print fully customizable scorecards. It will be very useful for unofficial events, as well as in various situations when it's necessary to print only one scorecard for a particular competitor

Screenshots:
image
image

@jonatanklosko jonatanklosko left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks great!

I didn't look at the UI code too closely, but as long as it gets the job done, it's all good :p

One UI nit: currently there is a lot of nested borders/boxes:

Image

We should kill at least one level of nesting. I think what we can do is to have each section as its own card (rather than all sections wrapped in a single card). The "Add event" button can be below all cards.

@jonatanklosko

jonatanklosko commented Jul 8, 2026

Copy link
Copy Markdown
Owner

(Also, I just pushed a fix, so there is a conflict in the scorecards code that needs to be resolved)

@maxidragon

Copy link
Copy Markdown
Contributor Author

We should kill at least one level of nesting. I think what we can do is to have each section as its own card (rather than all sections wrapped in a single card). The "Add event" button can be below all cards.

@jonatanklosko
image

@jonatanklosko

Copy link
Copy Markdown
Owner

Glorious :)

@jonatanklosko
jonatanklosko merged commit fd00237 into jonatanklosko:master Jul 8, 2026
5 checks passed
@maxidragon
maxidragon deleted the custom-scorecards branch July 8, 2026 14:02
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.

2 participants