Opened 10 years ago

Closed 10 years ago

#6341 closed Bug (fixed)

Mouse cursor on the editor area does not change to "insert icon".

Reported by: Senthil Owned by: Sa'ar Zac Elias
Priority: Normal Milestone: CKEditor 3.4.2
Component: General Version: 3.3.1
Keywords: Oracle Cc: Senthil

Description

when the mouse cursor hovers over the editor area the "pointer" icon does not change to the "insertion" icon. This behaviour is confsuing our customers and they think that the editor area is not accessible. Also we have noticed, FCKEditor 2.6.6 works fine for the same usecase.

Attachments (2)

6341.patch (471 bytes) - added by Sa'ar Zac Elias 10 years ago.
6341_2.patch (1.6 KB) - added by Sa'ar Zac Elias 10 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 10 years ago by Sa'ar Zac Elias

Status: newconfirmed

We must ensure that the cursor will still work correctly for objects like images and tables.

Changed 10 years ago by Sa'ar Zac Elias

Attachment: 6341.patch added

comment:2 Changed 10 years ago by Sa'ar Zac Elias

Milestone: CKEditor 3.4.2
Owner: set to Sa'ar Zac Elias
Status: confirmedreview

It's for some reason impossible on FF [perhaps a Bugzilla ticket should be filed]. The same usecase didn't work on FF with FCKeditor as well.

comment:3 Changed 10 years ago by Garry Yao

Status: reviewreview_passed

Actually Webkit is the only that works natively in the way that's expecting by this ticket, the patch doesn't resolve everything but at least give a more consistent style that looks like the desktop experience.

comment:4 in reply to:  2 Changed 10 years ago by Frederico Caldeira Knabben

Replying to Saare:

It's for some reason impossible on FF [perhaps a Bugzilla ticket should be filed]. The same usecase didn't work on FF with FCKeditor as well.

Please do not close this ticket without reporting it to Mozilla.

comment:5 Changed 10 years ago by Sa'ar Zac Elias

Status: review_passedreview

Fixed with [5903].
I found out that the problem is caused by the fact that the HTML height is not streched to 100%. Proposing a workaround here.

Changed 10 years ago by Sa'ar Zac Elias

Attachment: 6341_2.patch added

comment:6 Changed 10 years ago by Garry Yao

Status: reviewreview_failed

Did you see a problem with the your committed patch, for me that's enough.

comment:7 Changed 10 years ago by Sa'ar Zac Elias

Status: review_failedreview

With FF, if the content is shorter than the editor frame, the text cursor will not appear underneath it.

comment:8 Changed 10 years ago by Garry Yao

Status: reviewreview_failed

Why not making it a generic CSS change (for all browsers) and unify the 'autogrow' plugin accordingly?

comment:9 Changed 10 years ago by Sa'ar Zac Elias

Status: review_failedreview

Using generic CSS change brings regressions, it's safer just to have this change for FF.

comment:10 Changed 10 years ago by Garry Yao

Status: reviewreview_passed

comment:11 Changed 10 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: review_passedclosed

Post fixed by [5929].

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