Opened 13 years ago
Closed 6 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 13 years ago by
comment:2 Changed 13 years ago by
I was confusing the RichText? attributes with the AlwaysShowToolbar? plugin... two different things -- see #6400 for the latter
comment:3 Changed 12 years ago by
| Milestone: | 1.2 → 2.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 11 years ago by
| Milestone: | 2.0 → future |
|---|---|
| Owner: | liucougar deleted |
comment:5 Changed 10 years ago by
| Milestone: | future → 2.0 |
|---|
Not sure why I changed this to "future", 2.0 is the only chance to remove an attribute.
comment:6 Changed 6 years ago by
| Cc: | bill,alex → bill, alex |
|---|---|
| Resolution: | → wontfix |
| Status: | new → closed |
Actually, I don't think we'll port the Editor to 2.0 at all.
Note: See
TracTickets for help on using
tickets.
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?