feat: Added a focus parameter to the setMarkdown and setHTML functions

[azmeuk]

#1772 introduced a focus parameter to the moveCursorToEnd function.

Now I encounter the need to not take the focus when setting markdown in the editor, hence I added a focus parameter to the setMarkdown and setHTML functions, defaulted to true, that is passed to moveCursorToEnd.

The autofocus parameter introduced in #1772 is also passed to setMarkdown or setHTML during the editor constructor to ensure cursor is not taken by the editor if autofocus is false. I think this may help with #1945 and #1802, because without this patch, if the editor is initialized with data it still takes the focus.

I am not really sure how to test this so I did not write any, but please let me know if you think this is important.

ping @js87zz

[azmeuk]

@js87zz any chance that you might review this soon? Thank you in advance!

[js87zz]

@azmeuk
I'm sorry for the late response. First of all, I haven't considered adding the parameter to solve this issue, so I'm testing internally to see if there's a better way. I'll review it more and comment further. Thank you!

[seanwang1998]

@js87zz I'm currently using v3.1.0, setting autofocus to false, and still stealing focus after creating the Editor

[js87zz]

@azmeuk
I confirmed that autofocus is ignored only in markdown tab mode. Since there is the PR(#2085) for this, the problem can be resolved.
In addition, since users can control the focus through the cursorToEnd parameter in setMarkdown(or setHTML) API, it seems that there is no need to add parameter for this.
So unfortunately, the PR will not be merged. Thank you so much for taking the time to help resolve our issues. The autofocus option you added was very helpful to users. Thank you :)👍

[azmeuk]

No problem. As long as the bug gets fixed in the end. 👍