[Opera] Close inline style fails

[Garry Yao]

1. Start typing a few words within an empty page;
2. Click to open 'Bold' style, continue to type some words;
3. Click again on 'Bold' to close the style;
   • Actual Result: Selection is still inside the strong element.

Doesn't seems to be a regression.

To summarize, this TC is applicable for format buttons as well as inline styles from "Styles Dropdown", font names and font sizes - All inline styles.

Checkout comment 20 for more details

[Garry Yao]

Confirmed in Opera 11 Alpha/10.6.

[robmueller]

This isn't just an Opera problem. In Chrome (8 and 9) and Safari (5) changing the Bold/Italic/Underline style doesn't work at all unless you have some text selected. If it's just an insert cursor, nothing changes.

It all appears to work ok in Firefox and IE (8 and 9PP7).

[Garry Yao]

Ok, only happens when text selection is at the end of the block, which degrade it's priority.

[Hallvord R. M. Steen (Opera Software)]

it seems to work OK again in Chrome, leaving Opera as the only browser with the problem..

[Garry Yao]

Update: The impact of the bug is actually much wider than observed, as the following selection situations are also affected:

<p><strong>bold^</strong></p> // Ticket TC
<p><strong>^bold</strong></p>
<p><a href="#">link with <strong>bold^</strong></a> text</p>
<p><a href="#">link with <strong>^bold^</strong></a> text</p>

[Hallvord R. M. Steen (Opera Software)]

Since you're working on this at the moment, do you know what Opera does differently from, say, Chrome? If you could explain what the difference is, we could prioritise a fix on our side and you would not need to add further Opera sniffing to your code base!

(PS: sorry to have messed with your priorities and other state yesterday, just wanted to boost priority to get to this shortly myself.)

[Hallvord R. M. Steen (Opera Software)]

(BTW, does this patch change behaviour regarding #6874 ?)

[Frederico Caldeira Knabben]

Replying to hallvord@…:

Since you're working on this at the moment, do you know what Opera does differently from, say, Chrome? If you could explain what the difference is, we could prioritise a fix on our side and you would not need to add further Opera sniffing to your code base!

(PS: sorry to have messed with your priorities and other state yesterday, just wanted to boost priority to get to this shortly myself.)

I would love to see that, Hallvord. Here you have the issue at your side:
​https://bugs.opera.com/browse/CORE-38253

To workaround the problem, the patch proposed by Garry is appending and empty text node to the selection point (which I believe will be the fix at your side as well).

[Hallvord R. M. Steen (Opera Software)]

Thanks for the bug report. I'm right now testing an internal build with cursor-related changes, and the original problem seems to work OK in that build (without Garry's workaround)!

To be clear, how I'm testing is for example

• create a new document
• type 'hello ' (no quotes)
• press ctrl-b
• type 'world' (no quotes)
• press ctrl-b
• type ', this is me'

In the internal build, only the word 'world' will be bold. Since the new test case still fails, we probably have more than one issue..

[Frederico Caldeira Knabben]

I'm holding on this one at our side at this point, waiting for some conclusion at Opera's side.

[Frederico Caldeira Knabben]

#7802 has been marked as DUP.

[Hallvord R. M. Steen (Opera Software)]

testing with ctrl-b works fine, but for some reason the cursor goes to the start of the document when using the B button on the toolbar to toggle bold off. I'm not sure why ctrl-b and the button would behave differently, but I guess you need to restore the cursor position after clicks in the toolbar and this breaks something?

I don't think we'll get this fixed from our side in the near future, so if Garry's patch solves this please apply it.

[Jakub Ś]

This issue was duplicated in #8143.

@mike1 has proposed little different TC: Clear editor, turn on one of bold/italic/underline/strikethrough, type some text, press the same button again to turn off formatting.

Result - you can't turn off formatting (from CKE 3.0) and the cursor goes to the beginning of the text (from CKE 3.4).

[Hallvord R. M. Steen (Opera Software)]

Did you add a workaround? The real editor works better in some of our internal builds now, though the test still fails.

[Frederico Caldeira Knabben]

I don't think we have any workaround for it.

My current tests with Opera 11.52 (Build 1100) are even worst. On step 3, the caret moves to the start of the paragraph.

[Jakub Ś]

#8515 was marked as duplicate.