Skip to content

Add Collapsible Menu to Improve Usability and User Experience #2228

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

Closed
wants to merge 11 commits into from

Conversation

thecoder93
Copy link
Contributor

Before going further, I would like to have feedback, if a collapsible side menu can be useful to improve usability and user experience. The PR is in draft also because there are still some side effects (as you can see from the video) when you shrink the page with the monaco editor, still present on main branch, when in some cases you switch from samples to tutorials.
Feel free to close the PR if this is considered a "nice to have".

Registrazione.schermo.2025-03-13.alle.15.42.13.mov

@thecoder93
Copy link
Contributor Author

@microsoft-github-policy-service agree

@thecoder93
Copy link
Contributor Author

I also opened a new issue, because as I said, the problem is also present on main. #2231

@thecoder93
Copy link
Contributor Author

Fixed issue layout monaco editor with PR #2244

@thecoder93 thecoder93 marked this pull request as ready for review March 25, 2025 17:27
@minestarks
Copy link
Member

@thecoder93 I appreciate your contribution, and I'm sorry for not having given this feedback early on: At the moment, we consider this PR out of scope for how we envision the Q# Playground. The playground is mostly meant to be developer-facing (i.e. for us, the contributors of Q#) so we can test out changes to the product... as such we're prioritizing simplicity over user-friendliness to minimize maintenance.

So I'm going to go head and close this PR - but thank you for the work you put in.

@minestarks minestarks closed this Apr 21, 2025
@thecoder93
Copy link
Contributor Author

@minestarks Seems legit. Thank you for your time.

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

Successfully merging this pull request may close these issues.

2 participants