Opened 10 years ago

Closed 7 years ago

#1644 closed Task (fixed)

Proposal: remove references to cursor: hand; in css

Reported by: alfonsoml Owned by: alfonsoml
Priority: Normal Milestone: CKEditor 3.4
Component: General Version:
Keywords: Confirmed Review+ Cc:


Only IE 5.5 needs the "cursor: hand;" rule, IE6 and every other browser will understand properly the "cursor: pointer;"

By removing those rules we avoid some warnings in the error consoles about the styles.

Attachments (1)

1644.patch (2.2 KB) - added by alfonsoml 7 years ago.
Proposed patch

Download all attachments as: .zip

Change History (6)

comment:1 Changed 10 years ago by fredck

What about IE 5.5 compatibility? We haven't dropped it yet.

comment:2 Changed 10 years ago by alfonsoml

The only drawback for them would be that they won't see the hand cursor on some elements.

I think that they have greater problems (like the lack of patches since it was EOL some time ago and I doubt that there are too many developers dedicating a machine just to keep that ugly browser around, so their experience in the web must be already somehow unpleasant).

According to the percentage of IE5 (that means 5.0 and 5.5) is 1.5%, and it keeps going down. In a splitted comparison I think that the use of 5.0 is slightly higher than 5.5 because that's the default browser in W2000, so that leaves no more than 0.8% of users with that browser. Losing a hand cursor isn't a big problem from my point of view.

In fact, forcing them somehow to upgrade would be beneficial for everybody.

Changed 7 years ago by alfonsoml

Proposed patch

comment:3 Changed 7 years ago by alfonsoml

  • Keywords Review? added
  • Milestone set to CKEditor 3.4
  • Owner set to alfonsoml
  • Status changed from new to assigned

IE 5.5 isn't supported in CKEditor so there's no reason to keep the "cursor: hand" statements

comment:4 Changed 7 years ago by fredck

  • Keywords Confirmed Review+ added; Discussion Review? removed

comment:5 Changed 7 years ago by alfonsoml

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [5636]

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