Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@vmcj
Copy link
Collaborator

@vmcj vmcj commented Jul 18, 2025

In case someone provides logo's which are far from squarish the aspect ratio gets lost as the logo is stretched. As I'm not 100% how far we want to go we should just pick the right commits and squash those as the solution.

commit: Move image size to CSS (shows the problem):
image

commit: Keep original aspect ratio
image

commit: Center the logo over the cell
image
For the last one I think it only works when all cells get centered like this. Maybe there should just be a maximum on the lenght of the display name of the team to get a more consistent scoreboard.

@deboer-tim
Copy link
Collaborator

FWIW I haven't put a huge amount of effort into the CDS scoreboard as we never tell anyone about it so no real 'users'. I'm good with fixes/improvement, but would put more effort into something that would have external users, has a newer technical base, and potentially could be reused by other tools (e.g. maybe scoreboard in the team view I've been slowly building). Re the commits:

commit: Move image size to CSS

  • I'm always against anything that'll stretch logos. :)

commit: Keep original aspect ratio

  • +1

commit: Center the logo over the cell

  • Agree we could limit # of lines and centre everything, but not sure I'd like it / prefer to keep PR to 1 issue.

@vmcj vmcj force-pushed the fix_aspect_ratio branch from 718f46f to ada144f Compare July 21, 2025 14:25
This also moves the layout choices to CSS for consistency.
@vmcj vmcj force-pushed the fix_aspect_ratio branch from ada144f to 67a6d61 Compare July 21, 2025 14:26
@vmcj
Copy link
Collaborator Author

vmcj commented Jul 21, 2025

Removed the centering commit, removed the commit of code cleanup.

@vmcj
Copy link
Collaborator Author

vmcj commented Jul 21, 2025

FWIW I haven't put a huge amount of effort into the CDS scoreboard as we never tell anyone about it so no real 'users'. I'm good with fixes/improvement, but would put more effort into something that would have external users, has a newer technical base,

It was a simple fix, agree that it isn't the highest prio.

@vmcj vmcj merged commit dde9ec5 into icpctools:main Jul 21, 2025
4 checks passed
@vmcj vmcj deleted the fix_aspect_ratio branch July 21, 2025 14:59
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