HabitRPG / habitrpg-mobile

HabitRPG mobile application under development. Angular + PhoneGap
286 stars 100 forks source link

Notification advice #249

Closed paglias closed 9 years ago

paglias commented 10 years ago

I need an advice on how to show multiple notifications in the mobile app.

Actually only a notification can be show at time making it difficult to show multiple notificationswhen, for example, a drop occur or a level up.

The possibilities are:

Show notification 1 for 2s then hide it and show notification 2 for another 2s....

@lefnire @huaruiwu @lemoness @deilann @SabreCat @Alys

CrokinoleMaster commented 10 years ago

I think 2 seconds each

lemoness commented 10 years ago

Do you have any gif prototypes?

On Sunday, August 3, 2014, Huarui Wu notifications@github.com wrote:

I think 2 seconds each

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-50986806 .

paglias commented 10 years ago

:+1: will do! If I discover how to make them, otherwise a video Il 03/ago/2014 21:27 "lemoness" notifications@github.com ha scritto:

Do you have any gif prototypes?

On Sunday, August 3, 2014, Huarui Wu notifications@github.com wrote:

I think 2 seconds each

— Reply to this email directly or view it on GitHub < https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-50986806>

.

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51000222 .

lefnire commented 10 years ago

LICEcap

deilann commented 10 years ago

...not only will it create gifs, but as a bonus, it'll make your head itch psychosomatically every time.

On Sun, Aug 3, 2014 at 12:44 PM, Tyler Renelle notifications@github.com wrote:

[LICEcap](http://www.cockos.com/licecap/

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51000740 .

paglias commented 10 years ago

Great! You'll have them tomorrow morning (10-15 hours) Il 03/ago/2014 21:46 "deilann" notifications@github.com ha scritto:

...not only will it create gifs, but as a bonus, it'll make your head itch psychosomatically every time.

On Sun, Aug 3, 2014 at 12:44 PM, Tyler Renelle notifications@github.com wrote:

[LICEcap](http://www.cockos.com/licecap/

— Reply to this email directly or view it on GitHub < https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51000740>

.

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51000764 .

lemoness commented 10 years ago

@deilann: HAH! @paglias: looking forward to it!

On Sun, Aug 3, 2014 at 12:57 PM, Matteo Pagliazzi notifications@github.com wrote:

Great! You'll have them tomorrow morning (10-15 hours) Il 03/ago/2014 21:46 "deilann" notifications@github.com ha scritto:

...not only will it create gifs, but as a bonus, it'll make your head itch psychosomatically every time.

On Sun, Aug 3, 2014 at 12:44 PM, Tyler Renelle notifications@github.com

wrote:

[LICEcap](http://www.cockos.com/licecap/

— Reply to this email directly or view it on GitHub <

https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51000740>

.

— Reply to this email directly or view it on GitHub < https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51000764>

.

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51001074 .

paglias commented 10 years ago

@huaruiwu @lemoness @deilann @lefnire LICEcap doesn't support Linux, so I had to go with a vide :(

https://www.dropbox.com/s/0jaevp7ap2twocb/prova.mp4

It shows notifications one after another with 2 seconds between each (even though notification 2 is sent after just 100ms)

paglias commented 10 years ago

This one is with notifications stacked https://www.dropbox.com/s/dmk3xcujqx2f902/prova2.mp4

Animations are missing but is not wanted, will fix it

lemoness commented 10 years ago

Hm, I'm unable to view the second one?

The first one is clean, but I worry that the change might not be visible enough, leading people to miss the second or third notification. The animation will probably help with this.

On Monday, August 4, 2014, Matteo Pagliazzi notifications@github.com wrote:

This one is with notifications stacked https://www.dropbox.com/s/dmk3xcujqx2f902/prova2.mp4

Animations are missing but is not wanted, will fix it

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51048772 .

CrokinoleMaster commented 10 years ago

@paglias if it doesn't support Linux, it's not worth using :P I'm also unable to view the second.

paglias commented 10 years ago

I'm leaning toward a Mac for my next computer I think ;) sadly but I had too many problems with hardware and battery

I'll redo the second one when I'm back home in 1hr Il 04/ago/2014 19:44 "Huarui Wu" notifications@github.com ha scritto:

@paglias https://github.com/paglias if it doesn't support Linux, it's not worth using :P I'm also unable to view the second.

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51092499 .

CrokinoleMaster commented 10 years ago

@paglias NOT mac!! that's the dark side

paglias commented 10 years ago

@lemoness notifications for sure will help but the duration is the same as now

paglias commented 10 years ago

this is the second video https://www.dropbox.com/s/dmk3xcujqx2f902/prova2.mp4

CrokinoleMaster commented 10 years ago

@paglias I still like the first one

lemoness commented 10 years ago

I'm more of a fan of the second one, just because it looks more similar to what we do in the web app... but it may very well be too cramped for mobile, so I am willing to have the first one.

On Mon, Aug 4, 2014 at 12:44 PM, Huarui Wu notifications@github.com wrote:

@paglias https://github.com/paglias I still like the first one

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51107626 .

CrokinoleMaster commented 10 years ago

@lemoness Second one would create a problem if they check off like 5 things at once. could also limit it to something like maximum of 2 or 3 on the screen at once.

paglias commented 10 years ago

I'm thinking about related notifications like when you level up, having everything shown together (lvl up, exp, and gp) probably feels better.

But I'm not sold on one of them yet, so tomorrow will try to implement all the other missing notifications and see the effect.

Then the 6th I will leave for a week and being abroad I won't be able to do much except maybe reading emails. So I'll leave you both the implementations' code, otherwise e can delay for a week Il 04/ago/2014 22:58 "lemoness" notifications@github.com ha scritto:

I'm more of a fan of the second one, just because it looks more similar to what we do in the web app... but it may very well be too cramped for mobile, so I am willing to have the first one.

On Mon, Aug 4, 2014 at 12:44 PM, Huarui Wu notifications@github.com wrote:

@paglias https://github.com/paglias I still like the first one

— Reply to this email directly or view it on GitHub < https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51107626>

.

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51116704 .

paglias commented 10 years ago

Limiting the number of notifications at the same time is probably a good idea if we go with option 2 Il 04/ago/2014 23:09 "Matteo Pagliazzi" matteopagliazzi@gmail.com ha scritto:

I'm thinking about related notifications like when you level up, having everything shown together (lvl up, exp, and gp) probably feels better.

But I'm not sold on one of them yet, so tomorrow will try to implement all the other missing notifications and see the effect.

Then the 6th I will leave for a week and being abroad I won't be able to do much except maybe reading emails. So I'll leave you both the implementations' code, otherwise e can delay for a week Il 04/ago/2014 22:58 "lemoness" notifications@github.com ha scritto:

I'm more of a fan of the second one, just because it looks more similar to what we do in the web app... but it may very well be too cramped for mobile, so I am willing to have the first one.

On Mon, Aug 4, 2014 at 12:44 PM, Huarui Wu notifications@github.com wrote:

@paglias https://github.com/paglias I still like the first one

— Reply to this email directly or view it on GitHub < https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51107626>

.

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51116704 .

CrokinoleMaster commented 10 years ago

@paglias Alright sounds good. Where are you leaving to?

paglias commented 10 years ago

Croatia with some friends!

CrokinoleMaster commented 10 years ago

@paglias Sounds like fun! I've never been to Europe.

paglias commented 10 years ago

Oh that Dublin in your profile is California not Ireland :)

I hope so, first choice was Greece but the we waited too much and the flight got too expensive Il 04/ago/2014 23:30 "Huarui Wu" notifications@github.com ha scritto:

@paglias https://github.com/paglias Sounds like fun! I've never been to Europe.

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51120482 .

CrokinoleMaster commented 10 years ago

@paglias yes it always confuses people :P It's actually in SF bay area.

lemoness commented 10 years ago

Agreed that if we go with the second option, limiting them to 2 at a time is necessary. Interested to see how the combined notifications look!

On Mon, Aug 4, 2014 at 2:11 PM, Huarui Wu notifications@github.com wrote:

@paglias https://github.com/paglias Alright sounds good. Where are you leaving to?

— Reply to this email directly or view it on GitHub https://github.com/HabitRPG/habitrpg-mobile/issues/249#issuecomment-51118210 .

paglias commented 9 years ago

This has taken way more than expected but I'm finally getting there.

I decided that given the less space avalaible on the mobile app it would be better to unify at least stats notifications. So now we have one big notifications for: hp, mp, gp and exp

Out of it remains:

What do you think? I'm about unsure about the lvl + 1 and critical, if they should go together with the others but there would be probably too much info in a single notifications

paglias commented 9 years ago

If you want to try, (it's not styled, just text) the branch is paglias/notifications

CrokinoleMaster commented 9 years ago

sounds good. I'll check it out soon.

paglias commented 9 years ago

I've added style to stats notifications, what do you think?

https://www.dropbox.com/s/5ir8zavjftnt7dg/new-notifications.mp4?dl=0

CrokinoleMaster commented 9 years ago

very cool. Looks good to me!

paglias commented 9 years ago

Added styling for all the other notifications. I'm just not sure on the length of the timer. The actual one is 2.5s and seems a bit short to read drops and in fact it's quite less than half of the 7s we have on the website. But 7s would be too much for stats and the rest and when we have more than 1 notifications it blocks the user from using the upper part of the app.

So it's quite ready, missing only notifications for server errors which I'll add later

CrokinoleMaster commented 9 years ago

I think 2.5s is fine. Making it longer might be annoying because it covers the top.

paglias commented 9 years ago

PR in #275

lemoness commented 9 years ago

This is implemented, yes? Can I close it?