Last modified: 2014-02-27 23:35:43 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T63289, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 61289 - VisualEditor: Make parameters' entry expand to their contents, so lengthy parameter content isn't obscured
VisualEditor: Make parameters' entry expand to their contents, so lengthy par...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High minor
: VE-deploy-2014-02-27
Assigned To: Trevor Parscal
:
: 61859 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-13 01:04 UTC by WhatamIdoing
Modified: 2014-02-27 23:35 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description WhatamIdoing 2014-02-13 01:04:55 UTC
Actual Results:  
You can only see about 1.5 lines of text.  It should be tall enough to display two lines of text, even if people are using slightly larger fonts.


Reproducible: Always

Multiple complaints with screenshots:

https://en.wikipedia.org/wiki/File:Euromaidan_VisualEditor_infobox.PNG

https://en.wikipedia.org/wiki/Wikipedia:VisualEditor/Feedback/Archive_2013_13#Multiple_issues_-_minimal_space
Comment 1 James Forrester 2014-02-14 04:29:20 UTC
We may want to re-visit this when these boxes are rich content, but whilst as wikitext they're likely to be expansive more often than not.
Comment 2 WhatamIdoing 2014-02-18 19:15:30 UTC
Other suggestions from en.wp:
 
* At least three lines tall.

* Scrollbars as needed.

* Ability to resize the window vertically.

* Replace the grey area between parameters by a line (less space wasted, as readable as before), don't use large padding inside the fields, reduce white space between parameter name and value.
Comment 3 James Forrester 2014-02-21 18:59:20 UTC
As of Gerrit change #114527 parameter value boxes will now auto-size (without maximum), and when empty (or have a short value) will show as just a single line (actually "normal size" which is ~1.5x in height).

(In reply to WhatamIdoing from comment #2)
> Other suggestions from en.wp:
>  
> * At least three lines tall.

I think that's too large for default; most parameter values are simple entries (e.g. a file name or an author's surname). We want filling in the fields to seem like a quick, easy thing to do, not an overwhelming prompt to write an essay. However, we could possibly bump the minimum size to larger than 1.5x, though I'd still be worried about making them too large. Note the size of the boxes in the sketch of the citation dialog by way of example: https://www.mediawiki.org/wiki/VisualEditor/Design/Reference_Dialog#mediaviewer/File:VE-reference-dailog-design-04b.png – this change was done as part of that work. I suggest we re-visit this if it doesn't feel like it's working out very well.


> * Scrollbars as needed.

Right now they'll just grow without limit, which means if it's such an insanely large parameter value that it's larger than the dialog it will get scrollbars, but otherwise it will just grow. Maybe we could set a maximum size (10 lines? 5?) and then scroll when larger than that? Perhaps something for another bug?


> * Ability to resize the window vertically.

Which "window"? The dialog itself, or the text box? If the latter, that's now done; if the former, I don't think that that's a reasonable outcome (because it breaks the design for mobile, limited vision users, and others with small screen sizes, and because it's Yet Another Control for users to have to understand what it does, why they'd use it, and how it works).


> * Replace the grey area between parameters by a line (less space wasted, as
> readable as before), don't use large padding inside the fields, reduce white
> space between parameter name and value.

The first and third of these points is also being done as part of the citation dialog work; if anything, we're probably going to increase the padding on the input boxes in line with that work, though.


Am thus closing this bug as FIXED for now, with an adjustment to the title to reflect the design decision, and we can open further bugs for other ideas.
Comment 4 James Forrester 2014-02-24 19:46:01 UTC
*** Bug 61859 has been marked as a duplicate of this bug. ***
Comment 5 Drongou 2014-02-27 23:35:43 UTC
Cool, please keep no limit for the size, some template have a lot of content :
https://fr.wikipedia.org/w/index.php?title=Espèces_du_genre_Carex&oldid=96878307&veaction=edit

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links