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: passthrough to iconField has no effect. #7236

Closed
Mohamedemad4 opened this issue Sep 24, 2024 · 2 comments
Closed

Password: passthrough to iconField has no effect. #7236

Mohamedemad4 opened this issue Sep 24, 2024 · 2 comments
Labels
Resolution: Duplicate Issue has already been reported or a pull request related to same issue has already been submitted

Comments

@Mohamedemad4
Copy link

Describe the bug

When passing a pt to the Password component. I wanted to left align the iconField. So adding a pt that sets the className of IconField to p-icon-field-left should have done the trick.

Trouble is: It seems to have no effect. The style isn't even being passed to the icon field.
The reproducer also shows the same thing.

Reproducer

https://stackblitz.com/edit/vitejs-vite-1glvmw?file=src%2FApp.tsx

System Information

Need to install the following packages:
[email protected]
Ok to proceed? (y) y


  System:
    OS: Linux 5.15 Ubuntu 20.04.6 LTS (Focal Fossa)
    CPU: (8) x64 Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz
    Memory: 5.50 GB / 23.36 GB
    Container: Yes
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
    Yarn: 4.3.1 - ~/.nvm/versions/node/v18.18.0/bin/yarn
    npm: 10.8.2 - ~/.nvm/versions/node/v18.18.0/bin/npm
    pnpm: 8.15.6 - ~/.local/share/pnpm/pnpm
  Browsers:
    Chrome: 123.0.6312.105
  npmPackages:
    primereact: ^10.8.2 => 10.8.2 
    react: ^18 => 18.3.1 
    tailwindcss: ^3.4.1 => 3.4.9

Steps to reproduce the behavior

No response

Expected behavior

I expect that passing classes via pt will append them to the already existing classes.

@Mohamedemad4 Mohamedemad4 added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Sep 24, 2024
@Sumeet-2023
Copy link
Contributor

This same issue was opened by #6776

@melloware melloware added Resolution: Duplicate Issue has already been reported or a pull request related to same issue has already been submitted and removed Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible labels Sep 24, 2024
Copy link

This issue is a duplicate and has already been reported and possibly fixed. Please review other issues both open and closed for a similar issue to this one.

@melloware melloware closed this as not planned Won't fix, can't repro, duplicate, stale Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Duplicate Issue has already been reported or a pull request related to same issue has already been submitted
Projects
None yet
Development

No branches or pull requests

3 participants