Closed Bug 1046336 Opened 7 years ago Closed 7 years ago
.1-visual-refresh) System Sound Refresh
Replace the v.1 sounds with the v.2 design.
Summary: (2.1-visual-refresh) Sound Refresh → (2.1-visual-refresh) Media Sound Refresh
Component: Gaia::Camera → Gaia::System
Summary: (2.1-visual-refresh) Media Sound Refresh → (2.1-visual-refresh) System Sound Refresh
Patryk, Where are with this? Don't see any updates on this bug. Have we landed everything required for 2.1 w.r.t to new sounds? Sprint 3 ends tomorrow 8/29
No longer blocks: NewSounds-2.1
Attached are 3 improved system sounds: + homescreen: unlock + keyboard keys Kevin can we still get these into 2.1, I believe the risk is really low. + 3 sound files swapped out + ogg file extension renamed to opus If not can we push this into master?
I don't see any concern here given the impact of the content. But hope we can land these stuff before feature landing date next time. Ping Fabrice to see if he has any concern.
Flags: needinfo?(khu) → needinfo?(fabrice)
That's fine, feel free to land!
Attachment #8482360 - Flags: review?(timdream)
Comment on attachment 8482360 [details] [review] patch for Gaia/master Unfortunately bug 1060879 is in conflict with this patch. Since the key sounds is really really short, according to bug 964584 comment 20 the uncompressed WAV format is actually better in this use case (smaller file size and decoding effort). r=me, please convert key sound opus files to WAV before merging.
Attachment #8482360 - Flags: review?(timdream) → review+
Thanks :) Landed to master: https://github.com/mozilla-b2g/gaia/commit/a4011737faf631127f28d3168df79e4ebe7a5a5f
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Pavel, I don't see the special key sound being converted to WAV. Please push a follow-up commit to fix it. (this one: apps/keyboard/resources/sounds/special.opus)
Assignee: padamczyk → pivanov
Landed to master: https://github.com/mozilla-b2g/gaia/commit/cf67e067b58ffb4ff867a5187f95c28cdc2aa34b
If this is landing and merging, it needs the 2.1 tag. Adding.
feature-b2g: --- → 2.1
You need to log in before you can comment on or make changes to this bug.