yzeng25 commented on issue #1245: URL: https://github.com/apache/apisix-website/issues/1245#issuecomment-1236509255
Generally looks good to me. Just some thoughts: 1. Could you change the major background color to red/white/grey/blue/purple/orange-ish? The current dark green-ish background just does not match with the website's theme colors. 2. Move the "~~~" upward a bit, so that the APISIX logo and the lines are separated. 3. Eureka has its own logo, you can see this [search result](https://www.google.com.hk/search?q=Eureka&newwindow=1&sxsrf=ALiCzsZlGwSZgdvj-TonWsApPEIDX2xfXA:1662348605137&source=lnms&tbm=isch&sa=X&ved=2ahUKEwjPleiQ2_z5AhUH7jgGHb9zC9IQ_AUoAXoECAIQAw&biw=1728&bih=967&dpr=2#imgrc=0pZlaqPHw0vWcM&imgdii=ERxfBGb1ta-cQM), so I suggest replace the current spring logo with it. -- 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]
