AemieJ commented on pull request #442:
URL: https://github.com/apache/camel-website/pull/442#issuecomment-667066490


   @aashnajena  the reason to increase the font-size of the data is 'cause 
currently it looks all clustered one after the another. I provided the snapshot 
and I think if I land on the desktop I observe till the releases only on the 
first go. And the reason I am telling to not remove the box-shadow is 'cause if 
you observe in the mobile view, it just doesn't fit the theme it looks 
confusing and plain. The navbar pattern you're suggestion looks neat on the 
personal portfolio I am not sure about this site tho.
   
   Also, I observed and have suggestions for the project section. 
   * It would be nicer to have a consistent flow of how each section is 
structure i.e covered the same amount of horizontal section as the current 
website layout is. 
   * The width of the complete section could be altered to make the changes as 
in the snapshot.
   * Apart from that, I find it best fit to not assign a height to the images 
as they end up looking inconsistent (Camel-K image appears smaller than others) 
and keep a margin of 1 rem between the image and content.
   * The paragraph could be explicitly assigned a height so that the button to 
click appears at the same level for each project.
   
   
   
![project-section](https://user-images.githubusercontent.com/44139348/89017999-cb42bf80-d338-11ea-84d0-5af62d1dfdff.png)
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to