Ticket #3727 (closed Bug: fixed)
Cell Properties dialog missing color selection option
| Reported by: | damo | Owned by: | tobiasz.cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | General | Version: | |
| Keywords: | IBM Confirmed Review+ | Cc: |
Description
The cell border & background color options in the Cell Properties dialog do not provide an easy way for users to select colors.
FCKEditor 2.6 provides a color selector in this case.
Attachments
Change History
comment:1 Changed 4 years ago by tobiasz.cudnik
- Status changed from new to assigned
- Keywords Confirmed added
- Owner set to tobiasz.cudnik
comment:2 Changed 4 years ago by fredck
- Milestone changed from CKEditor 3.0 to CKEditor 3.1
We're not able to have this feature for the 3.0.
comment:3 Changed 4 years ago by tobiasz.cudnik
- Keywords Review? added
Patch reproduces color picker for table cells from FCKeditor. Most of old dialog's code had been reused.
Everything seems to work fine, but i have some doubts regarding cell property dialog layout. Right now it mimics V2 one, but it has some issues (check IE quirks) caused by new buttons. I'm proposing relocating them below text field.
comment:4 Changed 4 years ago by garry.yao
- Keywords Review- added; Review? removed
The patch is almost good for me, except:
- The color dialog matrix doesn't work in safari.
- Coding styles should be changed when migrating codes from v2.
comment:5 Changed 4 years ago by garry.yao
While it's always safe and prior to align with v2 features, I would vote for using float panel for this feature as Tobias mentioned, we could consider it after this got fixed.
comment:6 Changed 4 years ago by tobiasz.cudnik
- Keywords Review? added; Review- removed
Next patch fixes Safari 3 issue, which seems to have limit for table markup (?!), because it works with 3/4 of color table, but not with the whole one.
That's why i've delegated mouseover and click event handlers to table root, which reduced the markup and allowed safari3 to work.
I don't have this issue on safari4 (4.0 530.17 win).
