[GitHub] [camel-website] zregvart commented on issue #192: Made the logo shorter in shorter resolutions of screen

[GitBox <gi...@apache.org>]

zregvart commented on issue #192: Made the logo shorter in shorter resolutions of screen
URL: https://github.com/apache/camel-website/pull/192#issuecomment-597667563
 
 
   Preview is [here](https://builds.apache.org/job/Camel.website/job/PR-192/Preview/index.html).
   
   Nice work @psingh587, I think this is much better than what we have. I'm worried a bit about even smaller widths, I usually test with 360 and 411 widths in Firefox this profile is called "Galaxy S9/S9+ Android 7.0" and "Pixel 2 Android 8.0". It's not as nice with those.
   
   I'm not sure what is the smallest width that we should support, so it might be fine regardless.
   
   My opinion, and I don't have to be right, is that perhaps a combined approach with ideas from both @CSElonewolf and @psingh587 could be the winner here. Idea of putting the search in the menu is very interesting to me. In the long run I'd like the menu on smaller screens to take up the full width of the screen, that way we can increase the spacing and have larger touch targets.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services