Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#6962 closed Bug (fixed)

Opera: problems with "small hard sign" in russian: ъ

Reported by: alexanderilyin Owned by: fredck
Priority: Normal Milestone: CKEditor 3.6.1
Component: General Version: 3.5
Keywords: Cc: fredck, garryyao, hallvord@…


Decription in Russian and suggested solution for v.2 is here: and it's not working for 3.5

Problem is when user press "ъ" key, it's interpreted as CTRL+B (bold) combination so user could not input "ъ" in editor.

Attachments (1)

6962.patch (776 bytes) - added by fredck 6 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 6 years ago by hallvord@…

  • Cc fredck garryyao added
  • Status changed from new to confirmed

Is this a general, known problem? I would expect it to happen in all browsers, not just Opera, though I have not investigated.

This code is not guaranteed to work when users use keyboard layouts (such as Russian) that are typed directly (not through IME) and create keypress events for characters with Unicode value > 1000 (since 1000 is the lowest meta character constant used by CKEditor):

	getKeystroke : function()
		var keystroke = this.getKey();

		if ( this.$.ctrlKey || this.$.metaKey )
			keystroke += CKEDITOR.CTRL;

		if ( this.$.shiftKey )
			keystroke += CKEDITOR.SHIFT;

		if ( this.$.altKey )
			keystroke += CKEDITOR.ALT;

		return keystroke;

comment:2 Changed 6 years ago by hallvord@…

  • Cc hallvord@… added

comment:3 Changed 6 years ago by fredck

@hallvord, good observation. Do you think that fixing the constants to values over 10FFFF (the Unicode range top) would be enough at this point?

comment:4 Changed 6 years ago by hallvord@…

Certainly :). I don't expect any practical uptake of characters outside the current Unicode charts anytime soon..

Changed 6 years ago by fredck

comment:5 Changed 6 years ago by fredck

  • Keywords russian ъ removed
  • Owner set to fredck
  • Status changed from confirmed to review

Ok, so the fix should be that simple.

comment:6 Changed 6 years ago by garry.yao

  • Status changed from review to review_passed

comment:7 Changed 6 years ago by fredck

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

Fixed with [6916].

comment:8 Changed 6 years ago by fredck

  • Milestone set to CKEditor 3.6.1

comment:9 Changed 6 years ago by fredck

Packager file updated with [6817].

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