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

Chips: UX issue #3072

Closed
igo opened this issue Jul 17, 2022 · 0 comments · Fixed by #3073
Closed

Chips: UX issue #3072

igo opened this issue Jul 17, 2022 · 0 comments · Fixed by #3073
Assignees
@cetincakiroglu
Labels
Type: Bug Issue contains a defect related to a specific component.
Milestone
8.3.0

Comments

@igo
Copy link

igo commented Jul 17, 2022

Describe the bug

I noticed UX issue in Chips component. Users tend to type text without pressing enter or comma. Therefor value is not changed and typed text is not used.
Perhaps adding typed text as value on blur event could fix this issue.

Reproducer

No response

PrimeReact version

8.2.0

React version

17.x

Language

TypeScript

Build / Runtime

Next.js

Browser(s)

No response

Steps to reproduce the behavior

No response

Expected behavior

No response
@igo igo added Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible Type: Bug Issue contains a defect related to a specific component. labels Jul 17, 2022
cetincakiroglu added a commit to cetincakiroglu/primereact that referenced this issue Jul 18, 2022
@cetincakiroglu
Fixed primefaces#3072 - Chips: UX issue
215e1c9
@cetincakiroglu cetincakiroglu linked a pull request Jul 18, 2022 that will close this issue
Fixed #3072 - Chips: UX issue #3073
Merged
@cetincakiroglu cetincakiroglu self-assigned this Jul 18, 2022
mertsincan pushed a commit that referenced this issue Jul 18, 2022
@cetincakiroglu
Fixed #3072 - Chips: UX issue (#3073)
6d62f1d 
* Fixed #3072 - Chips: UX issue

* Add addOnBlur prop & update documentation
mertsincan added a commit that referenced this issue Jul 18, 2022
@mertsincan
Refactor #3072
fb62001
@mertsincan mertsincan modified the milestones: 9.0.0, 8.3.0 Jul 18, 2022
@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 Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cetincakiroglu cetincakiroglu

Labels
Type: Bug Issue contains a defect related to a specific component.
Projects
None yet
Milestone
8.3.0
Development

Successfully merging a pull request may close this issue.

Fixed #3072 - Chips: UX issue cetincakiroglu/primereact
3 participants
@igo @mertsincan @cetincakiroglu