[Improvement] Allow setBackgroundColor and setColor to be used without editor gaining focus about roosterjs

[JiuqingSong]

Can you explain more why you don't want editor to be focused?

[vhuseinova-msft]

For example I have a situation, when I want to change the default background color and text color. I need to set doNotAdjustEditorColor to true so everything isn't set back to defaults when the theme(dark/light mode) is changed.
But in this case, if the editor wasn't focused before, and I change the theme and use setBackground or setColor functions, the editor will be focused by default.
I think there also may be issues with focus in case if there are a few rooster js editors on the same webpage.

Just to share more info, it's not a blocking issue for me, just wondering if this functionality can be improved.

[JiuqingSong]

Do you mean you want to change background color of the editor root DIV?

[vhuseinova-msft]

These ones:

roosterjs/packages/roosterjs-editor-core/lib/corePlugins/LifecyclePlugin.ts

Line 14 in ac262d8

backgroundColors: {

[JiuqingSong]

Not quite understand. LifecyclePlugin is using the down level API setColor to change background color, not setBackgroundColor, they are two different thing.

[JiuqingSong]

setBackgroundColor should only be used by ribbon button, when user click on the button to change color of selected text. In that case we expect editor to be focused so user can keep typing.

It should not be used in other case.

For all other cases, you can use the down level API setColor instead.

[vhuseinova-msft]

I see, thanks for the clarifications! It would be great to have a link for setColor in the documentation for setBackgroundColor (or some other documentation in place) as sometimes it's not very easy to find this kind of documentation withou code checking.