Opened 10 years ago

Closed 10 years ago

#3579 closed Bug (duplicate)

Replace div samples applies the hover effect to all divs

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Low Milestone: CKEditor 3.0
Component: General Version: SVN (CKEditor) - OLD
Keywords: Cc:

Description

The :hover effect to show where the user can double click is applied to all the divs, and it can be slightly misleading.

The patch fixes this by changing the structure of the sample.html page so that the id "html" is applied to the container of the data and then restricting the css rules. That way it will work the same in the released version.

Attachments (1)

3579.patch (1.5 KB) - added by Alfonso Martínez de Lizarrondo 10 years ago.
Proposed patch

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by Alfonso Martínez de Lizarrondo

Attachment: 3579.patch added

Proposed patch

comment:1 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review? removed

Ops!

That breaks the toolbar[[BR]] I'm removing the review request, but this is a problem (the broken toolbar) that people are gonna report very soon.

The reset stylesheet will be broken as soon as anybody uses a rule based on IDs, so it should be reviewed or some warning be available in the documentation.

comment:2 Changed 10 years ago by Tobiasz Cudnik

Resolution: duplicate
Status: newclosed

DUP of #3775.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy