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

Dropdown: does not respect emitEvent false #16755

Open
andrii-dymov opened this issue Nov 15, 2024 · 1 comment · May be fixed by #16816
Open

Dropdown: does not respect emitEvent false #16755

andrii-dymov opened this issue Nov 15, 2024 · 1 comment · May be fixed by #16816
Labels
Status: Pending Review Issue or pull request is being reviewed by Core Team
Milestone
17.18.13

Comments

@andrii-dymov
Copy link

Describe the bug

Dropdown component emits value change event for it's corresponding control even so there is emitEvent: false parameter when I call setValue. It happens only if control has no initial value or it's value is null. Not sure if other components have the same issue.
It looks like conditions in allowModelChange method should be updated to fix this.
image

Environment

stackblitz

Reproducer

https://stackblitz.com/edit/stackblitz-starters-7jmis4?file=src%2Fmain.ts

Angular version

18.1.0

PrimeNG version

18.0.0-beta.4

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

18.20.3

Browser(s)

No response

Steps to reproduce the behavior

No response

Expected behavior

No response
@andrii-dymov andrii-dymov 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 15, 2024
@mertsincan mertsincan added Resolution: Help Wanted Issue or pull request requires extra help and feedback and removed Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible labels Nov 19, 2024
@mertsincan mertsincan added this to the Future milestone Nov 19, 2024
@github-actions GitHub Actions
Copy link

Due to PrimeNG team's busy roadmap, this issue is available for anyone to work on. Make sure to reference this issue in your pull request. ✨ Thank you for your contribution! ✨

mateuszstudentzero added a commit to mateuszstudentzero/primeng-zio that referenced this issue Nov 22, 2024
@mateuszstudentzero
Fix: Dropdown respects emitEvent=false when setting value (primefaces…
421ad63 
…#16755)
@mateuszstudentzero mateuszstudentzero linked a pull request Nov 22, 2024 that will close this issue
Open
@mertsincan mertsincan linked a pull request Nov 22, 2024 that will close this issue
Fix:16755 - dropdown emit event #16816
Open
@mertsincan mertsincan modified the milestones: Future, 17.18.13 Nov 22, 2024
@mertsincan mertsincan added Status: Pending Review Issue or pull request is being reviewed by Core Team and removed Resolution: Help Wanted Issue or pull request requires extra help and feedback labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Status: Pending Review Issue or pull request is being reviewed by Core Team
Projects
None yet
Milestone
17.18.13
Development

Successfully merging a pull request may close this issue.

Fix:16755 - dropdown emit event mateuszstudentzero/primeng-zio
2 participants
@andrii-dymov @mertsincan