Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Inconsistent behavior when clicking outside of a field #11246

Open
FelixMalfait opened this issue Mar 27, 2025 · 3 comments
Open

Inconsistent behavior when clicking outside of a field #11246

FelixMalfait opened this issue Mar 27, 2025 · 3 comments
Assignees

Comments

@FelixMalfait
Copy link
Member

For phones I have to press "enter" to valide my changes but for other fields it's saved automatically when I leave the cell.

cc @Bonapara what should we do? I think people expect fields to be saved on clickoutside?

There might be other impacted fields. Emails most likely works the same way.

@Bonapara
Copy link
Member

You're talking of when adding a secondary phone number right?

Image

I think it would feel weird to save the second number on click outside. To me, current approach makes sense since it's not just an input but a menu (unlike what we do for the first phone number).

@guillim guillim self-assigned this Mar 31, 2025
@guillim
Copy link
Contributor

guillim commented Mar 31, 2025

Taking this issue for this sprint, I think it would be a good idea to decide what to do.

In production, I see that the issue is both on main phones AND secondary phones

  1. For primary phones : shall we implement the save onclick outside ?
  2. For secondary we keep this current way ?

@Bonapara
Copy link
Member

  1. My bad, missed that we don't have autosave on click-outside for the first phone number. Let's implement it.
  2. Yes

@charlesBochet charlesBochet moved this from 🔖 Planned to 🏗 In Progress in 🎯 Roadmap & Sprints Apr 2, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Status: 🏗 In Progress
Development

No branches or pull requests

3 participants