fix(runtime): scoped slot append/prepend correct order after interaction

[yigityuce]

What is the current behavior?

GitHub Issue Number: #5969

What is the new behavior?

Documentation

Does this introduce a breaking change?

• Yes
• No

Testing

Other information

[yigityuce]

to summarize, I added s-sh flag to the existing prepend patch and copied that block (s-ol and s-sh updating block) to the appendChild patch as well.

the others (InsertAdjacentHTML, InsertAdjacentText, InsertAdjacentElement, append, etc) are using the prepend or appendChild internally, so they don't need to be updated

[christian-bromann]

@yigityuce do you think we can add a unit test for this change?

[tanner-reits]

@yigityuce Changes look good! Only request would be to add a test like @christian-bromann suggested. You could probably add a Webdriver test to our existing test suite for these methods here. Let me know if you have any questions!

[yigityuce]

hey @christian-bromann & @tanner-reits tests are added as a separate test file since to reproduce the issue, I need to remove the default slot wrapper and I didn't want to break the existing test case.

[tanner-reits]

@yigityuce Thanks! Everything looks good. I'll get this merged and try to get a release out this week!

[christian-bromann]

Released in Stencil v4.22.0