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

fix(PhoneNumber): ensure correct selection of value #2852

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

langz
Copy link
Contributor

@langz langz commented Nov 6, 2023

Added a test for the issue #2850

Not 100% sure about the actual fix yet 🤔
If someone else has a clue, please give it a go, and add a commit to this branch 🙇


Update:

Copy link

vercel bot commented Nov 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eufemia ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 8, 2023 0:33am

Copy link

codesandbox-ci bot commented Nov 6, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit a404005:

Sandbox Source
eufemia-starter Configuration

@langz langz requested review from tujoworker and henit November 6, 2023 13:37
@tujoworker tujoworker force-pushed the fix/phone-number-selection branch from 877dd36 to d17efb1 Compare November 7, 2023 17:20
@tujoworker tujoworker force-pushed the fix/phone-number-selection branch 2 times, most recently from d44b8d8 to 9a163ea Compare November 7, 2023 19:59
@tujoworker
Copy link
Member

We probably should split the two commits in separate PRs.
Also, added a sort/order to fix #2862.

What's changed? Now we process the whole list first when focusing instead of the first render.
This way, we skip processing it during SSR. Because its not visible anyway.

@tujoworker tujoworker force-pushed the fix/phone-number-selection branch from 9a163ea to 67ef8f2 Compare November 8, 2023 07:49
@tujoworker tujoworker changed the title fix(PhoneNumber): selection fix(PhoneNumber): ensure correct selection of value Nov 8, 2023
Copy link
Contributor Author

@langz langz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 🥇

@tujoworker tujoworker force-pushed the fix/phone-number-selection branch from 67ef8f2 to a404005 Compare November 8, 2023 12:18
@tujoworker tujoworker merged commit a20a764 into main Nov 9, 2023
7 checks passed
@tujoworker tujoworker deleted the fix/phone-number-selection branch November 9, 2023 08:07
tujoworker pushed a commit that referenced this pull request Nov 12, 2023
## [10.13.0](v10.12.0...v10.13.0) (2023-11-12)

### 🐛 Bug Fixes

* **Autocomplete:** ensure correct value selection during data change ([#2863](#2863)) ([d229ec2](d229ec2))
* **Autocomplete:** ensure on_change is firered during dataset update ([266ab6a](266ab6a))
* **PhoneNumber:** ensure correct selection of value ([#2852](#2852)) ([a20a764](a20a764))
* remove nullish operator to support Storybook v6 ([#2873](#2873)) ([f0bab63](f0bab63))
* typescript react type resolution in monorepos ([#2866](#2866)) ([44f04f3](44f04f3))

### ✨ Features

* **Accordion:** change sbanken expand icon to chevron ([#2854](#2854)) ([a7cb8b9](a7cb8b9))
* add "use client" to components that use client side only React APIs (support for RSC) ([#2874](#2874)) ([a84e962](a84e962))
* **Button:** change sbanken primary button text color to white ([#2855](#2855)) ([1571138](1571138))
* **Typography:** change sbanken bold to medium ([#2808](#2808)) ([4b0f930](4b0f930))

### 📝 Documentation

* **About:** add Snorre/Henrik/Ynge as contributors ([#2885](#2885)) ([fff72da](fff72da))
* **Autocomplete:** adds default value to keep_value props ([#2867](#2867)) ([7cae1fd](7cae1fd))
* fix font docs ([#2849](#2849)) ([13beec9](13beec9))
* **Ingress:** adds Ingress to docs ([#2879](#2879)) ([f771f3d](f771f3d))
* **Lead:** adds Lead to docs ([#2877](#2877)) ([0c0da7f](0c0da7f))
@tujoworker
Copy link
Member

🎉 This PR is included in version 10.13.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

(PhoneNumber): Sort country codes by display language
2 participants