Code review comment for lp:~trb143/openlp/media_state
Revision history for this message
| Phill (phill-ridout) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
L7953: I'm not convinced the user will want to see a dialog box every time the preview, go live, or edit a song that has audio....
Could you comment this out for no, I'mo concerned that this would too easily be overlooked. (Its only going to happen for people who try to load a song with background audio and have the media tab disable / problems with VLC integration.
If you can do that and resub, I'll be happy to approve it!