feat: Editor bar visibility (quince.master)

[vladislavkeblysh]

Description

• Fixed editor bar visibility when scrolling up

[openedx-webhooks]

Thanks for the pull request, @vladislavkeblysh! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[itsjeyd]

@vladislavkeblysh Thank you for this contribution!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (open-release/quince.master@eb127cd). Click here to learn what that means.

Additional details and impacted files

@@                      Coverage Diff                      @@
##             open-release/quince.master     #583   +/-   ##
=============================================================
  Coverage                              ?   92.43%           
=============================================================
  Files                                 ?      169           
  Lines                                 ?     3451           
  Branches                              ?      898           
=============================================================
  Hits                                  ?     3190           
  Misses                                ?      241           
  Partials                              ?       20           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[awais-ansari]

@vladislavkeblysh breaking changes have been mentioned here.

[vladislavkeblysh]

@awais-ansari I have seen it. I will fix it soon, thank you

[arbrandes]

Do we just need to press the merge button, now? Are all issues fixed?

[cmltaWt0]

Master PR: #582

[itsjeyd]

@arbrandes

Do we just need to press the merge button, now? Are all issues fixed?

There seem to be some issues left to fix on the master PR, so @vladislavkeblysh might need to update this PR and the Palm backport (#581) with some follow-up changes to get them ready for merge.

[vladislavkeblysh]

@awais-ansari @mphilbrick211 Fixed comment from master

[sundasnoreen12]

PR LGTM

[openedx-webhooks]

@vladislavkeblysh 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.