Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: #907 sidebar broken in playground #1066

Merged
merged 14 commits into from
Nov 4, 2024

Conversation

surajTade
Copy link
Contributor

@surajTade surajTade commented Oct 5, 2024

Description

Changes proposed in this pull request:

  • Corrected the CSS that was controlling the width of the sidebar

Related issue(s)

Fixes #907, #908

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@AceTheCreator
Copy link
Member

@surajTade can you share a video of your fix?

@surajTade
Copy link
Contributor Author

@surajTade can you share a video of your fix?

Screen.Recording.2024-10-05.224812.mp4

Copy link
Member

@AceTheCreator AceTheCreator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@surajTade i don't think you're doing the correct thing here 👀

@surajTade
Copy link
Contributor Author

@surajTade i don't think you're doing the correct thing here 👀

I tired that fix on different screen sizes and it seems to be working fine, can you please elaborate on what could be wrong in this?

@AceTheCreator
Copy link
Member

This is what i get when i test your implementation here:
https://github.com/user-attachments/assets/27c15026-6f77-42d5-96b3-5abda7984c28

@surajTade
Copy link
Contributor Author

This is what i get when i test your implementation here: https://github.com/user-attachments/assets/27c15026-6f77-42d5-96b3-5abda7984c28

I see, I'm not sure why I'm not facing this issue:

Screen.Recording.2024-10-06.100045.mp4

@AceTheCreator
Copy link
Member

@surajTade here's a draft PR that fixes this https://github.com/asyncapi/asyncapi-react/pull/1072/files. You can copy the fix from there :)

@surajTade
Copy link
Contributor Author

image
This is what happens when I apply your fix!

@AceTheCreator
Copy link
Member

image This is what happens when I apply your fix!

That's strange. Can i see your implementation? maybe update this PR with suggested changes

@AceTheCreator
Copy link
Member

@surajTade, removed the changes you made to the layout file. Kindly test it :)

@AceTheCreator
Copy link
Member

@surajTade any update?

Copy link

sonarcloud bot commented Nov 4, 2024

Copy link
Member

@AceTheCreator AceTheCreator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR ready to be merged

@AceTheCreator
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit 0247f8a into asyncapi:master Nov 4, 2024
10 checks passed
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 2.4.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sidebar broken in playground when preview switches to compact layout
3 participants