Merge lp://qastaging/~suutari-olli/openlp/combined-bible-quick-search into lp://qastaging/openlp
Status: | Superseded |
---|---|
Proposed branch: | lp://qastaging/~suutari-olli/openlp/combined-bible-quick-search |
Merge into: | lp://qastaging/openlp |
Diff against target: |
687 lines (+369/-87) 7 files modified
openlp/core/common/uistrings.py (+30/-0) openlp/plugins/bibles/bibleplugin.py (+6/-2) openlp/plugins/bibles/lib/biblestab.py (+56/-0) openlp/plugins/bibles/lib/manager.py (+26/-44) openlp/plugins/bibles/lib/mediaitem.py (+208/-41) resources/images/openlp-2.qrc (+1/-0) tests/functional/openlp_plugins/bibles/test_mediaitem.py (+42/-0) |
To merge this branch: | bzr merge lp://qastaging/~suutari-olli/openlp/combined-bible-quick-search |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Bentley | Needs Fixing | ||
Tomas Groth | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2016-05-02.
This proposal has been superseded by a proposal from 2016-05-02.
Description of the change
This branch introduces following improvements to Quick Bible search:
- Combined Reference/Text search which first performs the Reference
search and then moves to Text search if nothing is found.
- Added Search while typing functionality for Quick Bible search
- Possibility to use “.” when shortening Book names in Reference search.
For an example Gen. 1 = Gen 1 = Genesis 1.
- New/Improved error messages
(E.g. added actual example verses to Reference error)
- 3 New settings for controlling Quick search behavior
This branch also prevents users from performing Text searches which are:
- Shorter than 3 characters long (not including spaces)
- Searches consisting from only spaces
These currently possible bad search quarries result in LONG search times
and program instability/
separated by spaces, but that scenario is relatively rarer.
-------
In this re-proposal:
- Renamed one new function to better describe the action
- Removed 3rd empty line after imports
- Added Search while typing functionality for Quick Bible search
- Added a setting for controlling this behavior
- Refactored some code
lp:~suutari-olli/openlp/combined-bible-quick-search (revision 2642)
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
[←[1;32mSUCCESS
Please don't include the changed resources.py in the merge request, it makes it impossible to see the code that has actually been changed.
Places the new image in "resources/images/" if it is not already there, but during the test/review process don't include the updated resources.py. We'll generate it locally for testing.