caitpj opened a new pull request, #35564: URL: https://github.com/apache/airflow/pull/35564
When hovering over the Airflow logo in the top left, the pinwheel part of the logo cuts across the text part of the logo. This is (subjectively) poor design. In order to stop this from happening, while keeping the pinwheel animation, this PR has: - set text to transition slightly to the right when the animation activates. I have also set the pinwheel animation back to infinite. In my opinion, having read the [discussion](https://github.com/apache/airflow/issues/34019) that resulted in the change, it was not the correct decision to switch the animation from infinite over hover to 1.5 seconds. Another change was made, to allow users with a 'prefers-reduced-motion' preference to not have the animation playing, was the correct change, and did not need a further change to the length of time the pinwheel animates for. This is a design change, which is always subjective, so totally understand if review wants to bin this. Also sorry in advance for anything bad I've done; it's my 1st PR, I promise to do better next time. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
