Review: Needs Fixing Needs Fixing: Please add more detailed documentation to the Display and MainDisplay classes. First I thought you were just splitting the MainDisplay class (withought seeing the use of it). Later I saw that you are actually creating Display objects. I had to look through the diff to confirm my guess (that it is used to as preview windows).
Do not do this: The implementation of the Media Controller The Media Controller adds an own class for every API Currently these are QtWebkit, Phonon and planed Vlc. Do this instead: The implementation of the Media Controller. More text until the limit of 80 characters is reached. The Media Controller adds an own class for every API. Currently these are QtWebkit, Phonon and planed Vlc. It is much easier to read the docs in the code, when you do this. (There is a option in Eric which may be helpful. It changes the background color of the relevant characters when you go beyond the allowed character length.) Always end the sentence with a period. Look at this (periods missing): http://rst.ninjs.org/?n=1107f90fd81d51fbc803347489fc6a1b&theme=basic The signals names need to be fixed (line 933ff): "Media Stop" vs "seekSlider" vs "media_hide". -- https://code.launchpad.net/~crichter/openlp/media_rewrite/+merge/80503 Your team OpenLP Core is subscribed to branch lp:openlp. _______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : openlp-core@lists.launchpad.net Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp