ScoreBox fix (+ lacrosse icon mini fix)

[amjiao]

Overview

Fixes #80 and #66
Also fixes small bug with the logic of coloring the winning team's total score green.

Changes Made

• Fixed lacrosse icon
• Fixed border cutoffs by wrapping ScoreBox component in a Surface
• Fixed team name cutoff error by setting a max width of 70.dp (approximately how long "Cornell" is in the font size for > 4 data columns) to allow adequate space for both the team name column and all of the data columns (if there's < 4 data columns, then I allow the team name column to take up IntrinsicSize.Max to fill up empty space
• Fixed alignment issues by refactoring table to a Row of Columns structure as opposed to a Column of Rows.
• Added if-else statement in the total scores column so that the color is green if that team won
• Accounted for edge case where the number of data columns > 10, in which case the data columns become a LazyRow that scrolls between the fixed team name column and totals column

Test Coverage

• Wrote more test data for different game lengths (2, 4, 6, 10, 12 in total)
• appears correct in previews

Screenshots (delete if not applicable)

[amjiao]

(fix) I was basing my sizing estimates off of previews that were using fillMaxWidth, but there's start/end padding around the ScoreBoxes in the actual screen. Updated the thresholds where the sizing settings change to accommodate games with more data columns. (this seems like not best practice though because I'm picking .dp sizes based off of visual estimates, is there a better way to do this?)

Also, didn't end up using autosizing since each column is a different component so I was worried about text autosizing differently in each column.

[zachseidner1]

There are some minor code quality improvements to be made, but overall really nice job and clever approach!

[zachseidner1]

The only thing changing here is the period scores row. I wonder if we could just have the lazy and non lazy versions as separate composables, instead of having the entire table datas as separate composables? Lmk if that makes sense

[amjiao]

Tried this but ultimately had to keep the olde way because for the non-lazy version, the width of the TotalsColumn needs to be the same as each individual data column, so I had to construct the data row in the same component as the TotalsColumn so that I could set each column to have the same width.

[zachseidner1]

Nice, thanks for resolving the comments!