Opened 11 years ago

Closed 5 years ago

#6391 closed defect (wontfix)

RichText specifies height as attribute

Reported by: Adam Peller Owned by:
Priority: high Milestone: 2.0
Component: Editor Version: 1.1.0
Keywords: Cc: bill, alex
Blocked By: Blocking:

Description

This is inconsistent with Dijit. Shouldn't it be specified in CSS?

Change History (6)

comment:1 Changed 11 years ago by Adam Peller

add to that the minHeight attribute, which also mimics a CSS name, but with different meaning, I think, and the plugin name, which doesn't reflect the auto-expanding nature.

Given deprecation rules, perhaps the easiest thing to do is start over with a new plugin and deprecate this one?

comment:2 Changed 11 years ago by Adam Peller

I was confusing the RichText? attributes with the AlwaysShowToolbar? plugin... two different things -- see #6400 for the latter

comment:3 Changed 11 years ago by bill

Milestone: 1.22.0

Not sure if it's possible to change this but anyway, it would have to wait until 2.0 as it's an API change.

comment:4 Changed 9 years ago by bill

Milestone: 2.0future
Owner: liucougar deleted

comment:5 Changed 8 years ago by bill

Milestone: future2.0

Not sure why I changed this to "future", 2.0 is the only chance to remove an attribute.

comment:6 Changed 5 years ago by bill

Cc: bill,alexbill, alex
Resolution: wontfix
Status: newclosed

Actually, I don't think we'll port the Editor to 2.0 at all.

Note: See TracTickets for help on using tickets.