=== MacSlow is now known as MacSlow|lunch | ||
=== Estilanda_ is now known as Estilanda | ||
=== MacSlow|lunch is now known as MacSlow | ||
gventuri | nik90: I've updated the spec with vibrate settings | 15:12 |
---|---|---|
nik90 | gventuri: ah thnx | 15:12 |
nik90 | gventuri: there was one more question I had , but I will update the design query doc tonight | 15:13 |
gventuri | nik90: I've decided against having the vibration settings per alarm, it's a global setting | 15:13 |
nik90 | ok | 15:13 |
gventuri | nik90: what's about? | 15:13 |
nik90 | gventuri: about adding custom cities for the world clock | 15:13 |
nik90 | gventuri: in the spec, you wanted me to add a dialog where the user can enter a custom city name and choose the timezone. | 15:15 |
gventuri | nik90: cool | 15:15 |
gventuri | nik90: yp | 15:15 |
nik90 | gventuri: I wonder wouldn't it be better to search online when the city is not found in the local list we provide | 15:15 |
nik90 | gventuri: that's the behaviour we have in the old clock app | 15:15 |
gventuri | nik90: let me check | 15:17 |
gventuri | nik90: are you using an API? how does it work? | 15:18 |
nik90 | gventuri: yeha we are using an API which returns the timezoneID of the city the user searches for | 15:18 |
nik90 | gventuri: the time calculation is all done locally | 15:19 |
gventuri | nik90: that's beautiful | 15:19 |
nik90 | gventuri: this is also the same API that the Ubuntu Installer on the desktop uses | 15:19 |
nik90 | so it is reliable | 15:19 |
gventuri | maybe we could add visual feedback when searching online | 15:19 |
gventuri | thank you for pointing it out | 15:19 |
nik90 | exactly | 15:20 |
nik90 | yw | 15:20 |
gventuri | nik90: I'll give it some thought, you can ignore that screen then. I'll update the spec | 15:21 |
nik90 | gventuri: ok. I will until then implement other stuff like the fast scroll, swipe delete etc. | 15:21 |
nik90 | gventuri: btw how do you like the update? | 15:22 |
gventuri | nik90: have you got visuals for swipe to delete? | 15:22 |
nik90 | gventuri: no, but michal told me to implement the same as the one the address book uses | 15:23 |
nik90 | gventuri: sry, actually I got the visual spec | 15:23 |
nik90 | gventuri: I will be taking the work from the address book app to main consistency for Fast Scroll, Swipe Delete, Multiselect (alarm list) | 15:23 |
gventuri | nik90: actually have a look at the messaging app | 15:24 |
gventuri | they have a custom implementaion of swipe to delete | 15:24 |
* nik90 looks | 15:24 | |
nik90 | gventuri: did they release it? | 15:25 |
nik90 | gventuri: I am on the latest image and the swipe delete looks similar to what we already have in the clock app. The design spec shows a red variant | 15:26 |
gventuri | nik90: sorry forget it. Telephony is the right one | 15:26 |
gventuri | just spoke to mehow | 15:26 |
nik90 | yeah I will be borrrowing that from them | 15:26 |
gventuri | cool | 15:32 |
gventuri | nik90: there's a lag when you release 'Add city', until when you see the screen | 15:36 |
gventuri | maybe we could preload the screen so the rendering is instantaneous | 15:37 |
nik90 | gventuri: yeah it is taking too long to read the list of cities we provide. I am currently investigating on how to improve it. | 15:37 |
nik90 | gventuri: on the laptop it takes only 0.7 secs, while on the phone it is about 4.7 secs | 15:37 |
nik90 | gventuri: but we will fix that for sure | 15:37 |
gventuri | nik90: ok | 15:38 |
nik90 | gventuri: on more question. In the Phone spec at https://docs.google.com/presentation/d/1JvDyhsW17d1-Mz8OY1YMBKwfRI2z9qgyRjbujEsxEMk/edit#slide=id.g18895458d_00, we now show a snap decision. However as I told you Snooze will not be ready for RTM. So what buttons would like to be shown there? | 15:39 |
gventuri | nik90: that's great. Let me know when you want me to start reviewing it | 15:39 |
gventuri | aha | 15:39 |
nik90 | gventuri: at the moment, we show "Dismiss" and "Show" which both does the same thing | 15:39 |
gventuri | nik90: yeah it's a known problem | 15:40 |
nik90 | the snap decision takes a minimum of 2 actions. If we want only one of them to be shown, then we have to use a interactive notification similar to when receiving a sms. | 15:40 |
gventuri | nik90: exactly | 15:40 |
gventuri | nik90: we might watn to use a interactive notificaiton with a third line 'Tap to dismiss' | 15:41 |
nik90 | yeah that makes sense. | 15:42 |
gventuri | that would temporarily solve the issue | 15:42 |
gventuri | nik90: why can't we have snooze? I can't remember | 15:42 |
nik90 | so do we go with that solution for RTM? | 15:42 |
gventuri | nik90: ah, it's because there are no backgroudn apps right | 15:42 |
gventuri | nik90: app lifecycle | 15:42 |
nik90 | snooze requires implementation in the Ubuntu SDK and unfortunately the SDK devs are too busy with other stuff. So I was told it won't be ready for RTM | 15:42 |
gventuri | nik90: ok | 15:43 |
nik90 | well the datetime indicator (phone service) is the one which rings the alarm, so app lifecycle shouldn't be an issue. | 15:43 |
gventuri | teporarily I would use the interactive notification. It's not great but it's better than having 2 actions leading to the same place | 15:43 |
nik90 | ok | 15:43 |
gventuri | I'll update the spec | 15:43 |
* nik90 creates a bug report to track all of this | 15:43 | |
nik90 | thnx | 15:44 |
gventuri | nik90: Let me know when you want me to start reviewing the app | 15:44 |
nik90 | gventuri: If you want, you can start doing a quick light review. TBH about 90% of the functionality has been improved. | 15:45 |
nik90 | gventuri: s/improved/implemented | 15:45 |
nik90 | the only thing missing are the custom world city, swipe delete, multiselect and fast scroll. I will notify you when that batch of features is implemented. | 15:46 |
gventuri | nik90: cool | 15:46 |
gventuri | nik90: updated | 16:00 |
gventuri | nik90: let me know if you need something else | 16:00 |
gventuri | I'm off for today | 16:01 |
nik90 | gventuri: thnx. Will do | 16:01 |
nik90 | bye | 16:01 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!