Merge lp:~rharding/launchpad/bugfix_723417 into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Graham Binns |
Approved revision: | no longer in the source branch. |
Merged at revision: | 14331 |
Proposed branch: | lp:~rharding/launchpad/bugfix_723417 |
Merge into: | lp:launchpad |
Diff against target: |
12 lines (+1/-1) 1 file modified
lib/lp/app/widgets/textwidgets.py (+1/-1) |
To merge this branch: | bzr merge lp:~rharding/launchpad/bugfix_723417 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Graham Binns (community) | code | Approve | |
Review via email: mp+82577@code.launchpad.net |
Commit message
[r=gmb][bug=723417] Change default widget height of the DescriptionWidget to 15 vs the current 5.
Description of the change
= Issue Summary =
Moving to custom widget types removed the height setting on the field. The DescriptionWidget is used in this case and coded to 5 lines high.
== Proposed Fix ==
Change the height to 15 in the widget definition to improve the amount of visible space for entry.
== Pre-implementation Notes ==
It was brought up that it would be nice, especially in this case, to provide an auto expanding text widget. That idea and work is taking place over in: https:/
== Tests ==
There are no direct tests for checking height and since it's a potential moving target, none were added.
== Q/A ==
Start a new bug, confirm none of the existing are what you want to submit and view that the "Further information" comment input is now 15 lines high
Works for me. This is one of those things that can only be tested by QA (and even then relies on the happiness of others; YMMV).
Approved, anway.