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

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

Open
iskalyakin opened this issue Aug 6, 2021 · 7 comments
Open

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

iskalyakin opened this issue Aug 6, 2021 · 7 comments
Labels
Resolution: Cannot Replicate Issue could not be replicated by Core Team
Milestone

Comments

@iskalyakin
Copy link

iskalyakin commented Aug 6, 2021

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/stackblitz-starters-qdqnih2f

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/stackblitz-starters-qdqnih2f
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.

  • Angular version: 5.X
    18.0.1

  • PrimeNG version: 5.X
    18.0.1

  • Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
    all

  • Language: [all | TypeScript X.X | ES6/7 | ES5]
    all

  • Node (for AoT issues): node --version =
    all

@yigitfindikli yigitfindikli added the Status: Pending Review Issue or pull request is being reviewed by Core Team label Aug 6, 2021
@mertsincan
Copy link
Member

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
Copy link
Author

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

@mertsincan mertsincan reopened this Nov 11, 2022
@github-actions github-actions bot added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Nov 11, 2022
@mehmetcetin01140
Copy link
Contributor

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 open a new issue so we can include it in our roadmap?

Thanks a lot for your understanding!
Best Regards,

@iskalyakin
Copy link
Author

Hi! Unfortunately, the issue is still here: https://stackblitz.com/edit/stackblitz-starters-qdqnih2f

@mertsincan
Copy link
Member

Thanks a lot, @iskalyakin ! Could you please update issue template? version, sample link vs.
I'll set the correct milestone to this issue asap.

@mertsincan mertsincan reopened this Dec 16, 2024
@mertsincan mertsincan removed the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Dec 16, 2024
@mertsincan mertsincan added this to the 19.x milestone Dec 16, 2024
@github-actions github-actions bot added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Dec 16, 2024
@github-project-automation github-project-automation bot moved this to Review in PrimeNG Dec 16, 2024
@iskalyakin
Copy link
Author

ok, updated.

@mertsincan mertsincan modified the milestones: 19.x, 18.0.1 Dec 16, 2024
@mertsincan mertsincan removed the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Dec 16, 2024
@mehmetcetin01140 mehmetcetin01140 added Resolution: Cannot Replicate Issue could not be replicated by Core Team and removed Status: Pending Review Issue or pull request is being reviewed by Core Team labels Dec 17, 2024
Copy link

We're unable to replicate your issue. Could you please try it with the latest PrimeNG version? If the problem continues, could you create a reproducer using PrimeNG Issue Template or provide additional details by editing this issue? This issue will be closed if no activities in 20 days.

@mehmetcetin01140 mehmetcetin01140 modified the milestones: 18.0.1, 19.0.1 Dec 17, 2024
@mertsincan mertsincan modified the milestones: 19.0.1, 19.0.2 Dec 19, 2024
@cetincakiroglu cetincakiroglu modified the milestones: 19.0.2, 19.x Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Cannot Replicate Issue could not be replicated by Core Team
Projects
Status: Review
Development

No branches or pull requests

5 participants