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.

Sign up for GitHub

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

Jump to bottom

Password input actions won't be disabled when the input is #10084

Closed
JammingBen opened this issue Nov 29, 2023 · 1 comment · Fixed by #10096
Closed

Password input actions won't be disabled when the input is #10084

JammingBen opened this issue Nov 29, 2023 · 1 comment · Fixed by #10096
Assignees
@lookacat
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@JammingBen
Copy link
Contributor

Describe the bug

When disabling a password text input, the field itself will be disabled but the actions inside are still clickable.

Steps to reproduce

  1. Disable a password text input via :disabled="true"

Expected behavior

The buttons are greyed out like the input and can't be clicked.

Actual behavior

The buttons are not greyed out and can still be clicked.

image
@JammingBen JammingBen added Type:Bug Something isn't working Priority:p3-medium Normal priority labels Nov 29, 2023
@JammingBen JammingBen moved this to Prio 3 or less in Infinite Scale Team Board Nov 29, 2023
@lookacat lookacat self-assigned this Nov 30, 2023
@lookacat lookacat mentioned this issue Nov 30, 2023
Merged
9 tasks
@lookacat
Copy link
Contributor

We've decided to not display the buttons at all if the input is disabled.

@github-project-automation github-project-automation bot moved this from Prio 3 or less to Done in Infinite Scale Team Board Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lookacat lookacat

Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
Infinite Scale Team Board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bugfix: Remove password buttons when disabled owncloud/web
2 participants
@lookacat @JammingBen