izivkov / CasioGShockSmartSync

Apache License 2.0
112 stars 12 forks source link

Sort Calendar Events #44

Closed ahayes79 closed 1 year ago

ahayes79 commented 1 year ago

Is your feature request related to a problem? Please describe. Calendar events are in no discernable order and pull old events

Describe the solution you'd like Would like to have a way to sort events by date or alphabetical order. Would also like past events filtered out.

izivkov commented 1 year ago

OK, good idea. I'll try to get to it next week, after fixing a couple of issues.

It is not clear if you like to see old events or not. Currently they are not visible. By "filtered out", you mean, don't want to see them, right? If you do want to see them, that would be a problem, because it will clutter the UI with many events, and we only have room for a few on the screen. Also, would be confusing to the user, since they cannot set a reminder on the watch in the past.

ahayes79 commented 1 year ago

Hi, thanks for the quick response! No, I don't want to see old events. You're right, it does look like, with a couple odd exceptions, that old events are filtered out. The old dates I was seeing are the start dates of repeating events.

While you're working on it, here's a few other suggestions to look into when you have time:

izivkov commented 1 year ago

Hey @ahayes79 , you really like your Google Calendar :-). Frankly, I am not sure there is significant added value in implementing all these. This is a simple app, and can only handle a few events, all visible on one screen at the same time. Also, this is only a secondary feature of this app. The order of the events comes from the Google calendar, so maybe I can manipulate that, but I don't see a lot of values in allowing the user to select sorting further. In any case, these are good suggestions, but I have a lot of other issues to work on. Having said that, I will go over these carefully, and may implement some of them.

ahayes79 commented 1 year ago

Hi @izivkov Yes, totally realize that those suggestions are probably overkill for this app. Just some thoughts and any that you have time to implement would be great, but I completely understand that you've got other things going on. Appreciate your hard work on this!

izivkov commented 1 year ago

No problem. I'm away at the moment, will be back by the end of the week, and will take a look at this issue. As you said, it is probably not necessary to implement all these features, because on the all all you have to do is select the events you like to send to the watch. But I'll take a look and respond to this next week

On Mon., Jul. 31, 2023, 10:45 a.m. Andy, @.***> wrote:

Hi @izivkov https://github.com/izivkov Yes, totally realize that those suggestions are probably overkill for this app. Just some thoughts and any that you have time to implement would be great, but I completely understand that you've got other things going on. Appreciate your hard work on this!

— Reply to this email directly, view it on GitHub https://github.com/izivkov/CasioGShockSmartSync/issues/44#issuecomment-1658518801, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7M37UBYJPMJJ3TGWY7TJDXS7AJ5ANCNFSM6AAAAAA2NYISIQ . You are receiving this because you were mentioned.Message ID: @.***>

izivkov commented 1 year ago

Hey @ahayes79 . I looked into ways to sort the events. Currently, the events are sorted by the time of creation. It is possible to sort the events on the start date. This has some merit, but if you have repeating events, they will be sorted on the day of the first event, not on the next coming up event. So, for example of you set a repeating event last year, and the event has not expired, it will be displayed first. I would like to sort by next occurrence instead. All in all, since we have small number of events, and they are all visible on the screen, it does not make a lot of difference.

I may add sorting in the next release, but I don't want to complicate the UI by making it user-selectable. Let's just keep this as simple as possible.

I will close the issue for now. Thanks for your input, and do not hesitate to suggest ideas, or even contribute to the project.

izivkov commented 1 year ago

@ahayes79, I will add sorting by start date in the next release. Thanks for your suggestion.