Merge lp:~bladernr/checkbox/967457-remove-warning into lp:checkbox
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 1355 | ||||
Proposed branch: | lp:~bladernr/checkbox/967457-remove-warning | ||||
Merge into: | lp:checkbox | ||||
Diff against target: |
70 lines (+13/-7) 4 files modified
debian/changelog (+7/-1) qt/checkbox-qt.ui (+2/-2) qt/frontend/qtfront.cpp (+1/-1) qt/frontend/qtfront.ui (+3/-3) |
||||
To merge this branch: | bzr merge lp:~bladernr/checkbox/967457-remove-warning | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Checkbox Developers | Pending | ||
Review via email: mp+101293@code.launchpad.net |
Description of the change
Per discussion with Ara, as a workaround for bug 967457, this branch converts submissionWarni
I toyed initially on simply not displaying the label at all, and removing the label from the ui files and cpp file, but in doing that, the submission page looked pretty sparse and "unfriendly". This is a good compromise until we can add some bits in later that will show the warning message only when testing is incomplete.
Also, the warning message is a bit unnecessary before the end of testing because Sylvain Pineau's changes for bug #937715 make the submit and view buttons disabled until testing is complete anyway.