Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#11801 closed defect (fixed)

Editor: scrollbar too short, doesn't use full height

Reported by: bill Owned by: Jared Jurkiewicz
Priority: high Milestone: 1.6
Component: Editor Version: 1.5
Keywords: Cc:
Blocked By: Blocking:

Description

See test_FontChoice.html on FF (mac or windows). After fixing the missing semicolon in:

style="background-color: white; width: 100% height: 400px;"

becoming:

style="background-color: white; width: 100%; height: 400px;"

Then the scrollbar does not extend to the bottom of the Editor window. See attached screenshot.

Happens in 1.5 and trunk.

Attachments (1)

tooShort.gif (27.9 KB) - added by bill 9 years ago.
screenshot

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by bill

Attachment: tooShort.gif added

screenshot

comment:1 Changed 9 years ago by Jared Jurkiewicz

Editor doesn't take the height from the style line, it takes it from the height attribute. Setting height via style is problematic because of that old height attribute, which cannot be removed until 2.0, as it is a breaking change.

The testcase needs to be updated to remove that style and set it via the param

comment:2 Changed 9 years ago by Jared Jurkiewicz

Resolution: fixed
Status: newclosed

(In [22944]) Height is specified as an attribute, not a style, for editor. fixes #11801

comment:3 Changed 9 years ago by Jared Jurkiewicz

Note there is a bug to remove that and just use the style height: http://bugs.dojotoolkit.org/ticket/6391

Fixed the testcase(s)

comment:4 Changed 9 years ago by Jared Jurkiewicz

Milestone: tbd1.6
Note: See TracTickets for help on using tickets.