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

Set the left menu collapsable #441

Closed
M3lkior opened this issue May 26, 2021 · 23 comments
Closed

Set the left menu collapsable #441

M3lkior opened this issue May 26, 2021 · 23 comments
Labels
area/library Related to all activities around Library package enhancement New feature or request good first issue Good for newcomers Hacktoberfest Label issues as available for participants of https://hacktoberfest.digitalocean.com stale

Comments

@M3lkior
Copy link

M3lkior commented May 26, 2021

Reason/Context

Please try answering few of those questions

  • Why we need this improvement?

Depending on the screen resolution, the menu on the right side overlay the spec content

  • How will this change help?
    Allow the user to collapse this menu if needed
  • What is the motivation?
    A degraded user experience

image (13)

Description

Please try answering few of those questions

  • What changes have to be introduced?
    A new button to collapse the menu with a new collapsed style (a vertical layout to allow the expand ?)
  • Will this be a breaking change?
    no
  • How could it be implemented/designed?

A vertical layout with a collapse / expand icon

image

@magicmatatjahu
Copy link
Member

@M3lkior Hello! Thanks for the issue.

Depending on the screen resolution, the menu on the right side overlay the spec content

Do you mean left navigation, not right, yes? Also it's a strange. Could you described which version of template do you use? I see that you probably also use Chrome as browser. I'm using Chrome and Firefox and with 0.21.4 version of template I cannot reproduce the bug. Some resolutions from my spec in Chrome (in Firefox it looks similar):

Desktop:
image

Tablet:
image

Mobile:
image

About feature. Do you want to have something similar like SwaggerHub has, button to collapse left navigation like:

image

image

Am I right?

@M3lkior
Copy link
Author

M3lkior commented May 27, 2021

Hi @magicmatatjahu ; yes it is about the left menu, not the right,

  • Concerning the version of the used html-template, it is the 0.21.2
  • And for the proposal, it is exactly what i mean !

@magicmatatjahu
Copy link
Member

@M3lkior Thanks for quick response! Could you paste your spec to test? I am very concerned about why you have such a bug. Additionally, could you test the spec on version 0.20.0 of template? This is the pre-switch version to react component.

@M3lkior
Copy link
Author

M3lkior commented May 27, 2021

@magicmatatjahu i can not share the spec because of my enterprise specification (and because we are using avro definition in the spec) but i do a test with the v0.20.0 and the issue is not present. I can also reproduce with the v0.21.4

@magicmatatjahu
Copy link
Member

magicmatatjahu commented May 28, 2021

Honestly, I don't know what the problem could be. Apparently, as I see in your screenshoot, some styles stopped working for you and instead of (eg) flex the browser rendered it as fixed or absolute. It's hard for me to say what's wrong. At the moment, no one has reported a similar error. Maybe this is also a specification problem - which I doubt. It's really hard for me to conclude what is wrong.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity 😴
It will be closed in 60 days if no further activity occurs. To unstale this issue, add a comment with detailed explanation.
Thank you for your contributions ❤️

1 similar comment
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity 😴
It will be closed in 60 days if no further activity occurs. To unstale this issue, add a comment with detailed explanation.
Thank you for your contributions ❤️

@magicmatatjahu
Copy link
Member

@M3lkior Do you still have the problem with the hovering sidebar?

@M3lkior
Copy link
Author

M3lkior commented Sep 27, 2021

I test again, and the sidebar is now correctly shown but i think that the collapsible sidebar feature is a good ux feature :)

@magicmatatjahu
Copy link
Member

At the moment I am busy with something else, but I will mark this issue with Hacktoberfest label - someone should apply for this task :)

@magicmatatjahu
Copy link
Member

Moving to the asyncapi-react repository.

@magicmatatjahu magicmatatjahu transferred this issue from asyncapi/html-template Sep 27, 2021
@magicmatatjahu magicmatatjahu added area/library Related to all activities around Library package enhancement New feature or request good first issue Good for newcomers Hacktoberfest Label issues as available for participants of https://hacktoberfest.digitalocean.com labels Sep 27, 2021
@Krishks369
Copy link

Is this issue still active? if yes can you please once more elaborate the issue. I would like to contribute in it.

@M3lkior
Copy link
Author

M3lkior commented Nov 17, 2021

Hello @Krishks369 ; i think yes it is about the ui enhancement :)

@Krishks369
Copy link

I don't find any menu in right side of the UI
image

Please correct me if I am Wrong, i am a complete beginner

@M3lkior
Copy link
Author

M3lkior commented Nov 17, 2021

how do you obatin this screeshot ? if you use the html-template generation you will see the menu; if you look at this thread #441 (comment), there is a menu on the right side showing the operations name, the tags, the components, etc;

@magicmatatjahu
Copy link
Member

magicmatatjahu commented Nov 18, 2021

@Krishks369 Hi! If you are still willing to contribute, you need to make changes on the branch next, in the file https://github.com/asyncapi/asyncapi-react/blob/next/library/src/containers/Sidebar/Sidebar.tsx :

  • add some state where you will save expanded/collapsed state of menu
  • make it consistent with other styling

So you need to fork repo, switch to next branch and on it make changes. If you will need help, please ping me in this thread :) Every info, how to setup dev environment etc is in Readme.md

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Mar 19, 2022
@magicmatatjahu magicmatatjahu changed the title Set the right menu collapsable Set the left menu collapsable Jun 27, 2022
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Oct 26, 2022
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

This issue has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Mar 3, 2023
@sambhavgupta0705
Copy link
Member

Hey @magicmatatjahu can I work on this issue?

@github-actions github-actions bot removed the stale label Mar 14, 2023
@magicmatatjahu
Copy link
Member

@sambhavgupta0705 Sure, you can reuse code from #624

@sambhavgupta0705
Copy link
Member

hey @magicmatatjahu
I am having problem while setting up this repo locally
These are the errors:
image

@github-actions
Copy link

github-actions bot commented Aug 3, 2023

This issue has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Aug 3, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/library Related to all activities around Library package enhancement New feature or request good first issue Good for newcomers Hacktoberfest Label issues as available for participants of https://hacktoberfest.digitalocean.com stale
Projects
None yet
Development

No branches or pull requests

4 participants