Insert special character not keyboard accessible

[Damian]

This is the same issue as for the Smiley dialog. Ticket #3492 .

[Tobiasz Cudnik]

Tested on FF3, IE6, IE7, IE8.

[Artur Formella]

Review- because

• characters are not keyboard accessible in Google Chrome

• in IE7 and Safari arrows doesn't work at open (press tab to access)

• changes in smiley.js aren't related

[Tobiasz Cudnik]

Tested on FF3, IE7, Chrome2.

[Tobiasz Cudnik]

Second patch fixes:

• webkit compatibility for both Characters and Smiles dialogs
• IE7 needed delayed focus

[Martin Kou]

The logic should be ok. But can you change the links' style to display: block? It doesn't look good to have thin characters (e.g. the ! sign) highlighted by their outline.

[Martin Kou]

Still not ok - the background color of the individual cells is gone in IE. Also the width and height of the cells are incorrect.

[Martin Kou]

The problem should be caused by the modified attributes at the td cell generation code in L232 - L237. Also note that the href attribute at the td tag shouldn't be needed.

[Martin Kou]

Ok... almost there. But still one more issue.

The cell highlight style is incorrect at times. For example, if you keep on pressing tab and wait for the focus to go from the last cell to the Cancel button, and then go back to the first cell - you'll see that now we have two cells that are highlighted, the last and the first. This is wrong.

There should be a blur event which will cancel the highlight from keyboard. And also, I can see you're sharing the onMouse* functions for both mouse events and keyboard events. It would be better to rename them to say what the functions are doing in this case.

[Tobiasz Cudnik]

Ive refactored callback names to more describing and added missing events, which caused highlighting issues mentioned by Martin.

[Martin Kou]

Hmm.. it's still not fixed. It is still possible to have more than 1 cell highlighted like this:

1. Open special char dialog.
2. Press tab a few times.
3. Click on an empty area - the blurred cell is still highlighted.
4. Press tab a few more times.
5. Now you have two cells highlighted.

It seems to be me that it would be much simpler to hook the blur handler via onblur on the <a> tags, instead of adding it inside keyboard handlers.

[Tobiasz Cudnik]

This patch adds highlighted cell synchronization.

Using onBlur/Focus wont be as much simplification, because of mouse which still can highlight cells and it uses own events for that.

[Tobiasz Cudnik]

Fixed with [3664].