feat: Editor bar visibility (palm.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

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (dcb0f9e) 92.38% compared to head (5a10d7e) 92.40%.
Report is 5 commits behind head on open-release/palm.master.

Files	Patch %	Lines
...c/discussions/discussions-home/FeedbackWrapper.jsx	50.00%	1 Missing ⚠️

Additional details and impacted files

@@                     Coverage Diff                      @@
##           open-release/palm.master     #581      +/-   ##
============================================================
+ Coverage                     92.38%   92.40%   +0.01%     
============================================================
  Files                           163      162       -1     
  Lines                          3297     3278      -19     
  Branches                        890      883       -7     
============================================================
- Hits                           3046     3029      -17     
+ Misses                          232      230       -2     
  Partials                         19       19              

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

[e0d]

@vladislavkeblysh Looks like there are some coverage issues, can you have a look?

[vladislavkeblysh]

@e0d The coverage issues are not related to the new code added in this PR. It seems like this has been carrying over from previous PRs.

[itsjeyd]

@vladislavkeblysh Noted, thanks.

As I mentioned here, your reviewers will need to decide if that means the changes are good to merge as is.

[awais-ansari]

@vladislavkeblysh breaking changes have been mentioned here.

[vladislavkeblysh]

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

[mphilbrick211]

This should be ready to merge after the master is all set, @awais-ansari.

@vladislavkeblysh - there's a failing check here, would you mind rebasing to see if it fixes itself?

[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.