jscheffl commented on PR #35564: URL: https://github.com/apache/airflow/pull/35564#issuecomment-1819692172
> > > > I agree with Ryan, this was mainly an Easter Egg. I don't know if we want to mess with the logo too much. Can we double check everywhere else we use the Airflow SVG? > > > > > > > > > @bbovenzi I've had a look at the code base again and there is no other use of the logo as an SVG other than this one (in navbar.html). All the other uses of the logo use a png. > > > > > > Ok, let's just change the spacing between the pinwheel and the text then. I agree with Jed, I'm not a fan of the text shifting on hover. > > Damn... I knew I should never be a designer . Besides lagging the "Easter Egg" history I very much like the animation and would prefer to keep it. I am not a 100% designed but I treated the text shift as rather "cool" :-D But seems there are different opinions. The commit https://github.com/apache/airflow/assets/97813242/8f4f61d9-23ee-4553-a570-1ba758adb91a also looks good for me though. -- 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]
