StoDevX / AAO-React-Native

The St. Olaf community, now in pocket size.
GNU Affero General Public License v3.0
42 stars 16 forks source link

[course search] Cancel button in search bar only visible when focused #2771

Closed hawkrives closed 6 years ago

hawkrives commented 6 years ago

Follow-up to #2770

Idk.

hannesmcman commented 6 years ago

I think the third one could be a separate issue... especially since the absence of the cancel button when the search bar isn't focused is only an issue on iOS.

hannesmcman commented 6 years ago

Fixing the "cancel button not always visible" issue might have to go upstream to https://github.com/umhan35/react-native-search-bar

hawkrives commented 6 years ago

so stream of consciousness

what if

thoughts? elaboration requests?

hannesmcman commented 6 years ago

@hawkrives Not quite sure if I can picture what you're describing...could you elaborate?

hannesmcman commented 6 years ago

@hawkrives Can we rename this to: "Cancel button in search bar only visible when focused"?

hannesmcman commented 6 years ago

I've done some digging on this..and the showsCancelButton prop of react-native-search-bar does not help us here...my guess is that there is a native prop that needs to be bridged to in order to access the functionality we want. After looking at the docs for UISearchBar, it's not clear what this prop may be. However, it must be possible since the Apple Music app has a search bar with our desired behavior:

hawkrives commented 6 years ago

Mail.app does, as well, fwiw

hannesmcman commented 6 years ago

Anybody got a friend at Apple?

hannesmcman commented 6 years ago

Not a problem anymore, thanks to #2790.