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
feat: add other options to search events #758
Conversation
You forgot to pull latest commits from upstream. |
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.
Make sure that the travis build passes and please attach a screenshot whenever you make UI changes
@nikit19 pls review |
app/src/main/java/org/fossasia/openevent/general/search/SearchTimeActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fossasia/openevent/general/search/SearchTimeActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fossasia/openevent/general/search/SearchTimeViewModel.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fossasia/openevent/general/search/SearchTimeViewModel.kt
Show resolved
Hide resolved
Additionally a little more margin between the textviews in search fragment would look good IMO |
app/src/main/java/org/fossasia/openevent/general/search/SearchTimeActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fossasia/openevent/general/search/SearchTimeActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fossasia/openevent/general/search/SearchTimeViewModel.kt
Outdated
Show resolved
Hide resolved
@nikit19 done! |
@iamareebjamal pls review |
@iamareebjamal pls review |
@nikit19 The build is failing due to wildcard import. Pls review |
Fixes #733
Screenshots
* Since there is no event this weekend