search

etternagame / etterna

Advanced cross-platform rhythm game focused on keyboard play
https://etternaonline.com/
MIT License
474 stars 132 forks source link

Saturation fix #1289

Closed MarcusRanger closed 3 months ago

MarcusRanger commented 5 months ago

When you set the saturation of white (or any color with maximum value and minimum saturation), the color should still remain white because increasing the saturation should not have any effect unless the hue is defined and c.Value is less than 1. Also cleaned up ColorToHSV as it was possible for s = -nan (division by zero)

This addresses ticket #1285

poco0317 commented 5 months ago

didnt test but this will be merged into develop on the next push