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

Add toggle to grey out / disable panel in Side Navigation Component #1529

Open
bserrano1 opened this issue Sep 9, 2024 · 14 comments
Open

Add toggle to grey out / disable panel in Side Navigation Component #1529

bserrano1 opened this issue Sep 9, 2024 · 14 comments
Assignees

Comments

@bserrano1
Copy link

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is
We are currently asked to disable / grey out the R/Q panel when a user is viewing a historical entity.

Describe the desired solution
A clear and concise description of what you want to happen.
We are looking to see if there can be an added property to the NavigationLinks object that can allow disabling per panel as well as the possibility of hover over text to explain why a user cannot click on the panel.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
N/A

Additional context
Add any other context or screenshots about the feature request here.
image

@cwolf10
Copy link
Collaborator

cwolf10 commented Sep 19, 2024

Hey @bserrano1, just to clarify what you're looking for, would this feature be to disable individual links or just to disable the whole panel?

Also, is there a requirement at this point to display a tooltip when disabled? Or are you just anticipating that it might be a desirable feature in the future?

@bserrano1
Copy link
Author

Hello @cwolf10,
"would this feature be to disable individual links or just to disable the whole panel?"

  • It would be to disable the whole panel. For example, the Responsibility / Qualification panel would be greyed out and the user would not be able to interact / click it for routing.

Also, is there a requirement at this point to display a tooltip when disabled? Or are you just anticipating that it might be a desirable feature in the future?

  • Yes, there is a requirement to add a tooltip along with the disabled panel to explain to the user why they are unable to click on that specific panel.

@cwolf10
Copy link
Collaborator

cwolf10 commented Sep 24, 2024

@bserrano1 Last thing, do you have mockups of how the side nav should appear when disabled? Also is there one of the tooltip showing on the disabled sidenav?

@bserrano1
Copy link
Author

At this time, we have not been provided with any mockups.

@shannoncates-gsa
Copy link

@bserrano1 @cwolf10 the design team is waiting on the resolution of this ticket to move forward with mockups. We have never employed a tooltip on a grayed out nav-link before so we'd like to know the feasibility of it before we work on a new styling.

@cwolf10
Copy link
Collaborator

cwolf10 commented Sep 30, 2024

@shannoncates-gsa I don't see why this would not feasible. Hopefully that allows design to move forward with mockups.

As far as the resolution of this ticket, we can't really move forward on development without the mockups/Figma showing what the updated look and behavior should be.

@cwolf10
Copy link
Collaborator

cwolf10 commented Oct 10, 2024

@bserrano1 I am going to close this issue for now since there isn't any work for the design team to do at this point. Once there are mockups of how the sidenav should look when disabled and how the tooltip should look on the disabled sidenav feel free to re-open this issue and we can see about getting those updates made.

@cwolf10 cwolf10 closed this as completed Oct 10, 2024
@cwolf10 cwolf10 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 10, 2024
@Sunset05
Copy link

We have recently received updated mocks for how this should look. To re-open this issue do i need to create a new issue? or can we use this same ticket?

@cwolf10
Copy link
Collaborator

cwolf10 commented Oct 25, 2024

I just reopened this issue. Just attach those mockups and we can see about getting it into our backlog.

@cwolf10 cwolf10 reopened this Oct 25, 2024
@Sunset05
Copy link

Okay thanks, attaching below!

This mock shows the Responsibility/Qualification tab grayed and disabled.

image

@cwolf10
Copy link
Collaborator

cwolf10 commented Oct 25, 2024

Do you have a mockup that shows the tooltip when hovering over it? Or is that still a requrement?

@shannoncates-gsa
Copy link

shannoncates-gsa commented Oct 25, 2024 via email

@bserrano1
Copy link
Author

@cwolf10 just wanted to see if there was anything else that was needed before the work can begin.

@cwolf10
Copy link
Collaborator

cwolf10 commented Nov 26, 2024

@bserrano1 Not that I can think of. If anything comes up once development starts, I'll let you know.

@cwolf10 cwolf10 self-assigned this Nov 26, 2024
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

No branches or pull requests

4 participants