search

primefaces / primeng

The Most Complete Angular UI Component Library
https://primeng.org
Other
10.15k stars 4.54k forks source link

Password. When disabled, you can tab to it and type value anyway. #10483

Open iskalyakin opened 3 years ago

iskalyakin commented 3 years ago

I'm submitting a ... (check one with "x")

[x ] bug report => Search github for a similar issue or PR before submitting
[ ] feature request => Please check if request is not on the roadmap already https://github.com/primefaces/primeng/wiki/Roadmap
[ ] support request => Please do not submit support request here, instead see http://forum.primefaces.org/viewforum.php?f=35

Plunkr Case (Bug Reports) Please demonstrate your case at stackblitz by using the issue template below. Issues without a test case have much less possibility to be reviewd in detail and assisted.

https://stackblitz.com/edit/primeng-password-demo-100?file=src%2Fapp%2Fapp.component.ts

Current behavior Make password field disabled. After it component doesn't react on click, but can be focused and edited by tab.

Expected behavior Disabled password component shouldn't be accessable either by click or tab.

Minimal reproduction of the problem with instructions Minimal demo: https://stackblitz.com/edit/primeng-password-demo-100?file=src%2Fapp%2Fapp.component.ts Steps:

  1. Make password field disabled.
  2. Tab to it. You can, and it seems like an issue.

What is the motivation / use case for changing the behavior? Current behavior can lead to forms filling issues for users.

Please tell us about your environment: ubuntu 18.04, yarn.

mertsincan commented 1 year ago

Hi,

So sorry for the delayed response! Improvements have been made to many components recently, both in terms of performance and enhancement. Therefore, this improvement may have been developed in another issue ticket without realizing it. You can check this in the documentation. If there is no improvement on this, can you reopen the issue so we can include it in our roadmap? Please don't forget to add your feedback as a comment after reopening the issue. These will be taken into account by us and will contribute to the development of this feature. Thanks a lot for your understanding!

Best Regards,

iskalyakin commented 1 year ago

Hi. Thank you for the answer. Unfortunatly, it looks like this problem is still here, at least in 13 version (which is used for password component example). https://stackblitz.com/edit/primeng-password-demo-kaqf6r?file=src%2Fapp%2Fapp.component.html