Skip to content
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

segmentedControl for sorting goals on gallery #89

Closed
wants to merge 6 commits into from
Closed

segmentedControl for sorting goals on gallery #89

wants to merge 6 commits into from

Conversation

krugerk
Copy link
Contributor

@krugerk krugerk commented Jul 10, 2020

  • allow sorting of the goals directly from the gallery
    via the SegmentedControl
  • sort by preference is synced with one available in the
    app settings
  • can combine filtering (by slug) with quick sort via segmentedControl

@krugerk krugerk added STY Style / polish / CSS / pigsty / eyesore UVI Will count as a User-Visible Improvement labels Jul 10, 2020
@krugerk
Copy link
Contributor Author

krugerk commented Jul 10, 2020

In keeping up with the previous discussion on sorting goals (MR #58), where it was decided the three lists of goals should not share a common sort but rather the sorting of goals on the gallery should be closer to what one sees on the website, a segmented control has been introduced to allow "sorting by tapping column headers".
The sorting preference in the app settings and the selected segmented control are kept in sync and only used on the gallery.

(Unrelated but still noticeable are the flickering graphs, noted as Bug #43.)

@krugerk krugerk marked this pull request as ready for review July 10, 2020 07:58
@krugerk krugerk requested a review from andrewpbrett July 10, 2020 07:58
@krugerk krugerk marked this pull request as draft July 26, 2020 13:28
@krugerk
Copy link
Contributor Author

krugerk commented Jul 26, 2020

GUI could use a bit of tweaking with dark appearance (#94 ) and/or on iOS 13 (with its different appearance for segmented controls - for example the visible color behind the rounded corners of the segmented control)

@andrewpbrett
Copy link
Contributor

I like this idea, I think we should get some user feedback about it. I want to get 5.7 released first and then we can put this into a 5.8beta and upload to TestFlight.

@krugerk krugerk self-assigned this Aug 19, 2020
@krugerk
Copy link
Contributor Author

krugerk commented Aug 28, 2020

If the sort pref only is for the gallery and we use this segmented control instead, then there need not be a separate screen just for picking the sort criteria.

@krugerk krugerk added needs tests and removed STY Style / polish / CSS / pigsty / eyesore UVI Will count as a User-Visible Improvement labels Sep 10, 2020
 - allow sorting of the goals directly from the gallery
via the SegmentedControl
 - sort by preference is synced with one available in the
app settings
 - can combine filtering (by slug) with quick sort via segmentedControl
Considering the sorting preference is used only on the gallery and
the gallery now contains a segmented control to sort by the tap
of a control (where one can also see the sort order used), a
separate VC is not needed to setting the sorting order.
These are now computed properties.
When the user has not yet selected a preference, the gallery
now explicitly sorts by deadline.
This is done without setting the preference in UserDefaults
as doing so indicates that the user set that option.
@krugerk krugerk closed this May 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants