Opened 14 years ago

Closed 12 years ago

#219 closed defect (duplicate)

Editor save button, color picker deficiencies

Reported by: David Ascher <david.ascher@…> Owned by: alex
Priority: high Milestone: 1.1
Component: Editor Version: 0.1
Keywords: Cc: david.ascher@…
Blocked By: Blocking:

Description

The editor has a couple of buglets IMO:

  • the save button results in an closing of the richedit control. I would suggest that save & close be broken out into two separate buttons
  • the colorpickers need (IMO) another cell which corresponds to "no color", meaning clear out any span-specific color markup.

Change History (9)

comment:1 Changed 13 years ago by dylan

Milestone: 0.5
Owner: changed from anonymous to liucougar

need to apply these suggestions to Editor2

comment:2 Changed 13 years ago by dylan

Component: WidgetsEditor

comment:3 Changed 13 years ago by b at cce dot ac dot nz

I completely agree about the editor colorpicker - it needs the 'remove color' option. Also, if theres a current color set, then that should be highlighted in the palette (if theres an exact color match).

I'm hoping to the editor colorpicker in style dialogs.

I'm not certain about the save button - it depends on how you want to use the editor. Perhaps there should be a setting in the constructor for whether you want it to close on 'save' or just send the data somewhere and stay open.

comment:4 Changed 12 years ago by alex

Owner: changed from liucougar to alex
Status: newassigned

comment:5 Changed 12 years ago by bill

Milestone: 0.91.0

comment:6 Changed 12 years ago by bill

Summary: Editor feedbackEditor save button, color picker deficiencies

We don't have a save button on editor anymore (I suppose that the editor could be inside an InlinEditBox? to achieve that though). Actually the colorpicker isn't there either, but I'd like to add that back in. Not for 0.9 though.

comment:7 Changed 12 years ago by bill

Milestone: 1.01.1

comment:8 Changed 12 years ago by Adam Peller

now obsolete. See #5774 for ColorPalette? issue.

comment:9 Changed 12 years ago by Adam Peller

Resolution: duplicate
Status: assignedclosed
Note: See TracTickets for help on using tickets.