search

camdendotlol / topstersorg

Book and music charts
https://topsters.org
GNU Affero General Public License v3.0
18 stars 9 forks source link

Increase chart resolution #37

Open camdendotlol opened 7 months ago

camdendotlol commented 7 months ago

Chart items are currently 260px width/height. This was super high-res when I was developing on my 1080p monitor in 2021, but on my newer high-res screens I notice that smaller charts look kind of blurry when blown up on the screen. There's a tradeoff with file size and possibly render performance, but I can probably go up to 360 or 400.

Originally I planned to have this set as a constant, but it looks like it's hardcoded in one spot each here and on the topster package.

Edit: The topster package has been updated to allow a custom cellSize param.

Edit 2: The font size will need to be increased proportionally to the cell size, because otherwise the font will appear smaller as chart items increase in size. The default is 260px cells with 16pt font. 260 / 16 is 16.25, so a good ratio might be doing floor division like CELL_SIZE // 16.25.

camdendotlol commented 2 months ago

This might be realistic once #40, which drastically reduces the number of times the base canvas is rendered, is merged.