Merge lp:~suutari-olli/openlp/click-slide-to-go-live-from-blank into lp:openlp
Status: | Superseded |
---|---|
Proposed branch: | lp:~suutari-olli/openlp/click-slide-to-go-live-from-blank |
Merge into: | lp:openlp |
Diff against target: |
247 lines (+85/-6) 6 files modified
openlp/core/common/settings.py (+1/-0) openlp/core/lib/mediamanageritem.py (+1/-0) openlp/core/ui/generaltab.py (+7/-0) openlp/core/ui/slidecontroller.py (+25/-5) openlp/plugins/presentations/lib/messagelistener.py (+5/-1) tests/functional/openlp_core_ui/test_slidecontroller.py (+46/-0) |
To merge this branch: | bzr merge lp:~suutari-olli/openlp/click-slide-to-go-live-from-blank |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Bentley | Needs Fixing | ||
Raoul Snyman | Approve | ||
Tomas Groth | Pending | ||
Review via email: mp+300276@code.launchpad.net |
This proposal supersedes a proposal from 2016-07-16.
This proposal has been superseded by a proposal from 2016-08-10.
Description of the change
This branch introduces the functionality of unblanking
display from Blank to Black/Theme/Desktop for:
a) Clicking slide in "Live panel"
b) Next/Previous shortcuts (Green arrows)
c) Go to verse x.
d) When starting automatic playback (To end or Loop)
Also added "Unblank display when changing slide in Live" to advanced
options tab for disabling/enabling this behavior for a-c.
Additionally this branch also includes fix for bug
https:/
Do note that this branch does not fix this for Escape item blanking,
creating yet an another Escape exclusive bug.
-------
The only reason Escape item has been a good alternative for other
blank to methods is the functionality of resuming Live by clicking
slides and the fact it worked in single screen scenarios.
I can’t see any reason why it should not be removed after this branch
is merged since the single screen issue was already fixed earlier.
-------
Added 3 tests for checking display is re-blanked if it was
blanked before re-processing edited Live item.
Also fixed the issue where Next/Previous slide does not
unblank display for PowerPoint/Impress.
-------
Better fix for bug where display is unblanked on editing current live item.
This now sets a hidden setting to true while processing Live item and then changes it back to false.
Display is thus not unblanked at all during the process. (Old fix showed the edited slide for a small time)
Downside: All the new tests were based on the old
method and thus they were removed.
Fixed bug 1462420 (Double clicking preview adds items to service unlimited times)
- Added a hidden setting for controlling this behavior.
It is reset if any item is sent to preview from library.
- Sending the same item to service multiple times is still
possible by using the "Add button (icon)"
-------
- Added two tests for checking if doubleclicking preview should
add item to service or send it to live.
-------
lp:~suutari-olli/openlp/click-slide-to-go-live-from-blank (revision 2661)
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
Just tested a bit.
You have introduced the "Click live slide to unblank" setting, but a "Unblank display when adding new item" also exists in the general tab. You should probably move yours to be under the exiting one to keep similar settings in the same place. Currently your code doesn't honor the "Unblank display when adding new item", which it will have to do. As it is now the item goes live no matter if the setting is enabled or not.
Also the new setting should be "false" as default, this is new behavior, so users should enabled it if they want it.
Also added a code-comment below.