Merge lp://qastaging/~alisonken1/openlp/strings-templates2 into lp://qastaging/openlp
Status: | Superseded |
---|---|
Proposed branch: | lp://qastaging/~alisonken1/openlp/strings-templates2 |
Merge into: | lp://qastaging/openlp |
Diff against target: |
916 lines (+408/-397) 5 files modified
openlp/core/lib/htmlbuilder.py (+224/-226) openlp/core/ui/firsttimeform.py (+2/-2) openlp/plugins/bibles/forms/editbibledialog.py (+1/-1) openlp/plugins/songs/lib/mediaitem.py (+1/-1) tests/functional/openlp_core_lib/test_htmlbuilder.py (+180/-167) |
To merge this branch: | bzr merge lp://qastaging/~alisonken1/openlp/strings-templates2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Bentley | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2016-06-07.
This proposal has been superseded by a proposal from 2016-06-08.
Commit message
Convert htmlbuilder.py to use Template() strings for css
Description of the change
Convert htmlbuilder.py to use Template() strings for css
- Changed strings to use Template() instead of format() for css building
- Fix htmlbuilder strings tests
- Cleanup template variables and fix tests to match
- bug 1590386 - string format error in editbibledialog.py
- bug 1590475 - string format error in mediaitem.py
-------
lp:~alisonken1/openlp/strings-templates2 (revision 2678)
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
[SUCCESS] https:/
Looks good but one comment which is general across all the code.