linguini1 commented on PR #167: URL: https://github.com/apache/nuttx-website/pull/167#issuecomment-3916601667
A few suggestions: - Remove the toggle for the simple UI - If the sponsor logos are going to stop/have an outline when you hover over them, I feel like clicking on them should do something (like maybe taking you to their website?). I would just have the carousel scroll and not be interactive. - Add the NuttX LinkedIn and Hackster.io pages to the little connect with us section (and also the footer) - The 2025 international workshop has passed and should be replaced with the 2026 one - The NSH shell interactive widget shows 12.7.0 in the preview but is actually 12.4.0 on the demo, they should be the same - I would not mention the nuttx-website repo on the site. Users don't care about this repository, only the product code for the RTOS/apps bundle - The text on the articles & publications page on my machine looks quite small. I think this page could also use a little more polishing to be cohesive with how good the releases page looks now - Again for the "key features" section, the boxes responding to hover makes it feel like they should be clickable. I would remove the animation there - The documentation button right under the title doesn't work on your demo (I assume it will on the real site)? - The stars button should link to the stargazers section for the repository - None of the navigation bar dropdowns have anything in them When you built your self-hosted demo, was it done using the same NuttX CI build process? As in, are you sure what is hosted on your demo will appear the same way when built via the nuttx-website build process? -- 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]
