argoproj / argo-rollouts

Progressive Delivery for Kubernetes
https://argo-rollouts.readthedocs.io/
Apache License 2.0
2.8k stars 877 forks source link

Letter "L" doesn't work after navigating to a rollout details page #2273

Closed Cylix closed 1 year ago

Cylix commented 2 years ago

Checklist:

Describe the bug

Search inputs on the UI do not accept the letter "L" after navigating to a rollout details page.

The bug only seem to affect the letter "L". It is probably due to a shortcut implementation since the letter "L" seems to be used for navigating back to the dashboard root page.

To Reproduce

  1. Navigate to the Argo Rollouts Dashboard
  2. Try to input the letter "L" in the namespace or rollout search bars: it works πŸ‘
  3. Navigate to a Rollout Details Page
  4. Try to input the letter "L" in the namespace or rollout search bars: it doesn't work anymore 🀯

Expected behavior

We should be able to input the letter "L" in search inputs.

Screenshots

Version

v1.2.1+51c874c

Logs

No logs, this is a UI issue. There are no logs in the browser console.


Message from the maintainers:

Impacted by this bug? Give it a πŸ‘. We prioritize the issues with the most πŸ‘.

dcortright commented 2 years ago

@zachaller I would like to take this up for Hacktoberfest.

zachaller commented 2 years ago

Sounds good and thanks for jumping in. Feel free to reach out if you need anything!