Merge lp:~rharding/launchpad/bugfix_723417 into lp:launchpad

Proposed by Richard Harding
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
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://bugs.launchpad.net/launchpad/+bug/891735

== 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

To post a comment you must log in.
Revision history for this message
Graham Binns (gmb) wrote :

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.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/app/widgets/textwidgets.py'
2--- lib/lp/app/widgets/textwidgets.py 2011-02-09 23:57:52 +0000
3+++ lib/lp/app/widgets/textwidgets.py 2011-11-17 17:50:29 +0000
4@@ -273,7 +273,7 @@
5 class DescriptionWidget(TextAreaWidget):
6 """A widget to capture a description."""
7 width = 44
8- height = 5
9+ height = 15
10
11
12 class NoneableDescriptionWidget(DescriptionWidget):