spreed
spreed copied to clipboard
fix(MediaSettings): sync preferences with devices direct selection
☑️ Resolves
- Fix #12185
- Fix #12120
🖌️ UI Checklist
🖼️ Screenshots / Screencasts
| 🏚️ Before | 🏡 After |
|---|---|
🚧 Tasks
- [x] Do not show "Apply Settings" on changes that touch devices only.
- [x] Update preferences needs to be called
@update:deviceIdevent. This will cover both media settings and Talk settings devices changes. - [x] To move "check if preferences update is called at least once" to
showModal(), because user may open and close media settings without any pref update and without reload. It makes more sense to also force it next time they open it. - [x] Split audio and video preferences update
🏁 Checklist
- [ ] 🌏 Tested with Chrome, Firefox and Safari or should not be risky to browser differences
- [ ] 🖥️ Tested with Desktop client or should not be risky for it
- [ ] 🖌️ Design was reviewed, approved or inspired by the design team
- [ ] ⛑️ Tests are included or not possible
- [ ] 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required