argoproj / argo-workflows

Workflow Engine for Kubernetes
https://argo-workflows.readthedocs.io/
Apache License 2.0
15.13k stars 3.21k forks source link

The navItems are not clickable with vimium (accessibility) #12813

Open michaelfresco opened 8 months ago

michaelfresco commented 8 months ago

Pre-requisites

What happened/what did you expect to happen?

solution

The nav-bar items are not clickable with Vimium. In the screenshot above I included a fix for it. If you add: role="checkbox" it becomes clickable!

Version

latest

Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.

Logs from the workflow controller

N/A

Logs from in your workflow's wait container

N/A
michaelfresco commented 8 months ago

https://github.com/argoproj/argo-ui/blob/4848ea1a2f6fa4e34b72fb84ff0ee91453dfbe26/src/components/nav-bar/nav-bar.tsx#L39

I think that if we add role="checkbox" to that line, it would work.

                        <div className={classNames('nav-bar__item', { active: isActiveRoute(locationPath, item.path) })}
                            onClick={() => context.router.history.push(item.path)} role="checkbox">
                            <i className={item.iconClassName}/>