Skip to content
This repository has been archived by the owner on Nov 2, 2021. It is now read-only.

Fixes the weird placeholder strings appearing when language was changed #2510

Closed
wants to merge 1 commit into from

Conversation

sbhrwlr
Copy link

@sbhrwlr sbhrwlr commented Jul 4, 2021

Fixes #2507

A brief description of what changes your PR introduces. The better this description, the quicker we can review, exchange feedback, and merge!
After changing language the placeholder was displaying weird strings and not the cities suggestion you added it was happening because of useCallback function that starts running and do not terminate the priveous instance of useCallback I added language dependency to useCallback and function will terminate whenever the language state changes.

Fixes #2507

✔️ Compiles and passes lint tests
✔️ Properly formatted
✔️ Tested on desktop
✔️ Tested on phone

Before Fixing Screenshot :

initially
Initially wrong strings

After Fixing Screenshots

now

nowtwo
:

@suryavirkapur
Copy link

lgtm :)

@sbhrwlr sbhrwlr closed this Jul 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

After changing language and then coming back to english, the search placeholder displays weird strings
2 participants