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

[ux] No Visual Indicator for Scrollable Dropdown Menu #784

Closed
1 of 3 tasks
mullender opened this issue Mar 1, 2021 · 0 comments · Fixed by #792
Closed
1 of 3 tasks

[ux] No Visual Indicator for Scrollable Dropdown Menu #784

mullender opened this issue Mar 1, 2021 · 0 comments · Fixed by #792
Assignees
Labels
bug Something isn't working

Comments

@mullender
Copy link

Before submitting a bug please read: https://zwave-js.github.io/zwavejs2mqtt/#/troubleshooting/bug_report

Version 2.0.0

Build/Run method

  • Docker
  • PKG
  • Manually built (git clone - npm install - npm run build )

zwavejs2mqtt version: 1.1.0
zwavejs version: 6.5.0

Describe the bug

The controller actions dropdown menu is not obviously scollable
It took me looking at the page source code to figure out that there is a 'firmware update' action in the dropdown menu. After I found that I discovered that the dropdown was scrollable.

It would be nice if there was some indication that the dropdown was scrollable. Perhaps if the bottom most action was only partially visible (clipped at the bottom) it would become immediately clear there are more options below.

Anyway not a big deal.

image

@mullender mullender added the bug Something isn't working label Mar 1, 2021
robertsLando added a commit that referenced this issue Mar 2, 2021
robertsLando added a commit that referenced this issue Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants