fix: dynamically adjust dropdown width to prevent text wrapping#504
fix: dynamically adjust dropdown width to prevent text wrapping#504DevanjoyDas wants to merge 2 commits intoasyncapi:masterfrom
Conversation
There was a problem hiding this comment.
Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.
✅ Deploy Preview for peaceful-ramanujan-288045 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
ashmit-coder
left a comment
There was a problem hiding this comment.
LGTM! the workflow seems to be failing due to the year changing from 2024 to 2025.
|
hey @thulieblack should we change the year on the main page to 2025 or merge the pr's if only this test is failing? |
Yes @ashmit-coder , The error is occurring because the test "Contains correct heading" for the landing page heading is expecting the text: I have created PR #506 implementing the second solution. Root Cause of the Error:In the file https://github.com/asyncapi/conference-website/blob/master/cypress/e2e/Landing.cy.js Possible Solutions
Current Code Proposed New Code
Current Code Proposed New Code
|
|
cc @ashmit-coder, your review here would be appreciated 🙏🏽 |
|
@DevanjoyDas, can you resolve the conflict in your code? |
|
Closing this PR due to inactivity |
Description
This PR addresses the issue where the dropdown menu in the navigation bar had a fixed width, causing text wrapping for longer items and excessive space for shorter items. The dropdown width now adjusts dynamically based on the length of the longest link, ensuring a more consistent and responsive UI.
Screenshot

Fixes #503