Githawk: Search filter for inbox

Created on 4 Oct 2017  路  9Comments  路  Source: GitHawkApp/GitHawk

Easy way to search for stuff. No server search, just filter local stuff.

Filter for:

  • Title
  • Number
  • Owner
  • Repo


Bug Report Dump (Auto-generated)

Version 1.11.0 (1943)
Device: iPhone 7 Plus (iOS 11.0.1)
TestFlight: false

馃啎 feature request

All 9 comments

Can I pick this one up?

@weyert sure!

Sent with GitHawk

I am working on this issue at the moment. I was wondering what the best place would be to put the UISearchBar? Do we want to search query to be depending on the selected segment or apply to whatever we have selected?

Currently, thinking of creating a new section controller and add the UISearchBar in there and then filter the viewModels in the func objects(for listAdapter: ListAdapter)-method with the search query.

Also should the search query to match in all fields to show up or just any of them?

Sounds pretty solid to me. I think starting with just filtering the current selected items is fine.

Sent with GitHawk

I have created a first version of the search, now it will filter out items from viewModels array when they don't meet the search criteria. If the search query is numeric we search by the issue number.

Some questions, are you happy with the searchQuery-property or should it be wrapped into a view model similar SegmentControlViewModel which only contains the query?

@rnystrom @Sherlouk Search in Inbox

Image

Search in Issues:
Image

What's with the white sim? 馃槷

I like it this way! If you create a /AppleInternal directory you get an Internal-menu in the simulator where you can enable the color of the bezel. You can even customize the chrome color!

Sent with GitHawk

Gonna tweak the UI a little so its a bit more consistent, but this looks great

Was this page helpful?
0 / 5 - 0 ratings

Related issues

BasThomas picture BasThomas  路  3Comments

BasThomas picture BasThomas  路  3Comments

rnystrom picture rnystrom  路  3Comments

rizwankce picture rizwankce  路  3Comments

BasThomas picture BasThomas  路  3Comments