276593 - cmd_paragraphState vs setParagraphMode w.r.t. focus change

Status: NEW

(Core :: DOM: Editor, defect, P5)

Description

[Eyal Rozenberg]

I don't think calling

doStatefulCommand('cmd_paragraphState', /*whatever*/);

should change the focus to the message body. That is a side-effect not at all
implied by the command name.

So I tried using editor.setParagraphMode() instead, which worked, but didn't set
the paragraph mode string in the drop-down list until I changed the focus to the
message body.

Comment 1

[Eyal Rozenberg]

Hmmm... it seems that at endTransaction it says that 

"beginUpdate must be called before endTransaction is called."

were you referring to the lack of beginUpdate() calls? Because We have lots of
other code which does updates within begin/endTransaction() and does not call
beginUpdate, yet it works. As it should, I believe. Either way, if beginUpdate()
is needed, then the logStringMessage can't give the updated value until the
entire operation has been performed... right?

Comment 2

[Eyal Rozenberg]

Uh, sorry, commented in wrong bug.

Comment 3

[Kathleen :Brade]

So calling editor.setParagraphMode() doesn't affect focus but doStatefulCommand
does?  Hmmmm.... did you find where the "focus()" call is?  I'm curious.

Comment 4

[Eyal Rozenberg]

> did you find where the "focus()" call is?  I'm curious.

No... lots of spaghetti code in the editor :-(

Comment 5

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.