Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add offline search functionality for addresses #35045
Add offline search functionality for addresses #35045
Changes from 91 commits
a3de03d
dcb0939
ec559a9
d1f00b6
f4b3233
3927555
254abbd
bffc960
45bcc82
113eff3
46200f8
f84f234
879ee6a
3d2c7ce
09f670c
21c204d
5bf583e
ffbef6c
1badfc0
529a57b
4760494
8e37883
fd79182
13a07cd
fd5978b
cec47a1
3d212a4
b1f668f
0aa04aa
ad6ab06
715d987
e03748c
60e2285
2f02fdd
2421ec8
3d36e3e
af8fae1
463d185
232df83
3ee5a28
01725ca
32882eb
1d2ba32
14d35c9
8749419
3cff20c
4bd2ae6
c135670
aa9e2c5
4e262bc
93f48ae
882d5c7
6fd9754
86398c6
82e5ca0
4e4fe79
d6284d2
8837fe0
7330834
76a7fda
7a05d93
2cf661c
8f1e5a1
1c5c0f7
87c3265
2740a6b
6337b57
3ecc869
af15403
d842db6
3be71e1
4dac414
3003023
8907eef
d4d9883
557cf6d
08c9a5d
4da5ffe
eda219c
b6b5741
a3dbfef
8c21175
baf70a9
0c38e44
7d58b3a
16b8027
c8dce42
3aef2b9
580af72
97642c8
cb4eaeb
1693acd
9b74216
a116369
553cd86
c2de6ef
6fdb7a8
e34aaa2
32eb193
eb3175c
16f5d89
1097761
86905c4
18610b4
7446682
9f70527
ee38c7e
94dda99
3c2d677
3ce2ddf
0301900
8468261
f7195c0
49e9755
ae0ce2d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I'm missing something, could you elaborate on why we need initial values (why do we use
useRef
?)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eVoloshchak
useRef
is used here to store the initial value of window height, so that it can be compared with the current window height value to determine if the virtual keyboard is shown or not. AboutisSmallScreenWidthRef
I had to use a ref too because the callback I pass to theDimensions.addEventListener
cannot have dependencies, otherwise multiple event listeners would be set 😄There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is duplicated in both files, we can move this outside of
useSubmitButtonVisibility
by using it directly inIOURequestStepWaypoint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eVoloshchak
Good point, but I think it would be better for this style to be usable across components. I could extract this into a separate function that receives isSubmitButtonVisible as an argument and returns the style, what do you think?