← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

Re: [Merge] lp:~mzanetti/ubuntu-clock-app/drop-audioRole into lp:ubuntu-clock-app

 

W dniu 22.10.2015 o 14:52, Jim Hodapp pisze:
> Review: Needs Information
> 
> I would like to better understand why you're dropping the role completely instead of moving to the NotificationRole? As it stands right now, it'll use the old MultimediaRole at the media-hub and PulseAudio layer. So if you have music playing in music-app and then you go into the clock-app to preview the alarm sound, it should actually pause the background music instead of the duck-in effect. If that's what you desire then this is fine.

See the alternative proposal from the description.

In any case, what you describe does not work today, the preview plays
over music of reduced volume.

I don't think it should be Notification (or current - alert) anyway,
it's an alarm, after all? The counter argument is that if you want
Silent mode to keep the alarm preview silent, but then you can't change
the actual alarm volume with hardware buttons while playing the preview.
And you shouldn't be able to, or you will think the volume you're
changing is alarm, when in fact, it's the ringtone/alert volume.

W dniu 22.10.2015 o 15:15, Bartosz Kosiorek pisze:
> Review: Needs Information
>
> For me it even better, to pause current music and play preview alarm.
> It will let me, better adjust the volume of the alarm.
>
> I would like to pause audiobook, when alarm starts ringing. I will
> not miss anything from my audiobook. It will be annoying for me to
> rewind audiobook everytime alarm starts ringing.

Sure, that's a goal, but we can't get that today.

Also "every time alarm starts ringing", how many *alarms* (not
reminders) do you have set up?


-- 
https://code.launchpad.net/~mzanetti/ubuntu-clock-app/drop-audioRole/+merge/275179
Your team Ubuntu Clock Developers is subscribed to branch lp:ubuntu-clock-app.


Follow ups

References