vmiklos / plees-tracker

Plees Tracker is a simple sleep tracker for your Android phone.
https://vmiklos.hu/plees-tracker/
MIT License
161 stars 37 forks source link

FR : ability to rate & add info/comments to sleep items to better understand your sleep #161

Closed GLLM closed 3 years ago

GLLM commented 3 years ago

Hi !

I'm trying to monitor my sleep, and your app being nice and FOSS, I've used it for some nights. I'm happy with it ! :)

My problem is that, after a few nights, I cannot remember if, when waking up in the morning, I felt fantastic, good, tired or wrecked !

There are a number of info I'd be willing to add/store with each sleep item:

What would be lovely would be to be able to add more info to sleep items, to try detect patterns like :

As a MVP, simply adding & storing a sleep rating (from 0 to 5 stars) would be fine, so one is able to look it up afterwards. And make one's own stats/analysis.

As a V2, showing for each rating : the average bedtime, avg sleep length, avg room temp, avg moon phases, avg food quantity (1to5) would be fantastic.

What do you think about this ?

Thanks GLLM

vmiklos commented 3 years ago

Did you see that rating is already possible, just disabled by default? See https://github.com/vmiklos/plees-tracker/blob/master/doc/README.adoc#preferences-activity

GLLM commented 3 years ago

hi @vmiklos ! I'm kind of stupid, I did not see this :-/ However, I have feedback :

My other suggestions remains ;-)

Thanks GLLM

vmiklos commented 3 years ago

Let's try to focus on the rating itself here, ideally one issue is one request. :-)

I don't use the rating myself, so I don't have a strong opinion on that. The rating feature was added in https://github.com/vmiklos/plees-tracker/issues/28, @Narcolapser any opinion on the above? I.e. the request is to make the main activity rating widgets read-only, and only rate in the sleep activity. I'm a bit unsure, the current UI makes it easy to rate, which would be more complicated. On the other hand, it's currently confusing that a swipe results in a rate or on a sleep delete, depending on where you swipe, which is not ideal.

GLLM commented 3 years ago

Sure, on request at a time :) I was giving you an insight on the whole idea !

GLLM commented 3 years ago

Can I help with the info needed @vmiklos ? Is it end-user info, or dev info (for whihc, unfortunately, I wont be able to do much) ! thanks

GLLM commented 3 years ago

@vmiklos

(...) The rating feature was added in #28, @Narcolapser any opinion on the above? I.e. the request is to make the main activity rating widgets read-only, and only rate in the sleep activity. I'm a bit unsure, the current UI makes it easy to rate, which would be more complicated. On the other hand, it's currently confusing that a swipe results in a rate or on a sleep delete, depending on where you swipe, which is not ideal.

I confirm, it happened to me this morning. I was swiping in my sleep items, and inadvertently put my finger on a star strip as I had to scroll quite a bit and boom : I changed a sleep item's rating. I really think the rating should be readonly on the main screen ;)

vmiklos commented 3 years ago

Let's wait a week to see if @Narcolapser has anything to say. If we hear no opposition, then I can certainly go ahead with the rad-onlyness.

Narcolapser commented 3 years ago

It is indeed something I considered when you first implemented it for me. I was happy enough to get some one to do this for me for free that I didn't press the issue. But yes, the rating should be within the record and read only on the main screen. It would be nice if it pops up a little window asking how your slumber was when you clock out, but if I have to open the record just made to edit the score, I'm fine with that.

GLLM commented 3 years ago

Indeed, a small pop up appearing right after the user clocks out would be great !

vmiklos commented 3 years ago

Some find popups annoying, but making the rating bar read-only + move the editing of that to the sleep activity is certainly an option, I'll look at that.

GLLM commented 3 years ago

Some find popups annoying, but making the rating bar read-only + move the editing of that to the sleep activity is certainly an option, I'll look at that.

Great ! I'd vote to use a bit smaller stars as well ;)

vmiklos commented 3 years ago

The above does the main -> read only, sleep -> read-write change + doc update.

Regarding the size of the stars, https://stackoverflow.com/questions/6153587/how-can-i-decrease-the-size-of-ratingbar seems to show how to do this, but I'm a bit hesitant to go with that, considering this really depends on the size of your finger... I bet there is a good reason why the default size is what it is.

Similarly for the popup, this can be somewhat annoying. Hopefully the zero stars is a good indicator that you need to rate that item, without disturbing you with an explicit interruption.