feat: Enhancements to page

[vladislavkeblysh]

• Fixed font family on page
• Fixed borders on action bar on mobile view
• Fixed newest first text overlap another text
• Fixed displaying user name on mobile view

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

@jmakowski1123 With this set of changes I'm not sure if we need to step through the full product review process. It seems like they are just fixing some small visual inconsistencies. What do you think?

• Palm backport: feat: Enhancements to page (palm.master) #576
• Quince backport: feat: Enhancements to page (quince.master) #578

CC @mphilbrick211

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (67d79cb) 92.41% compared to head (009ccce) 92.41%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #577   +/-   ##
=======================================
  Coverage   92.41%   92.41%           
=======================================
  Files         169      169           
  Lines        3468     3468           
  Branches      900      900           
=======================================
  Hits         3205     3205           
  Misses        243      243           
  Partials       20       20           

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

[itsjeyd]

@jmakowski1123 Friendly ping about this question -- do you think this PR needs a round of product review?

[itsjeyd]

@jmakowski1123 @awais-ansari It sounds like product review for this PR has already started (?). If so, could you please point me to where that conversation is happening? It doesn't seem like we have a feature ticket for this PR, so it's hard for me to tell what its current status is.

CC @mphilbrick211

[awais-ansari]

@mphilbrick211 Is this PR still in a product review? or is it ready for merge?

[sundasnoreen12]

PR LGTM

[mphilbrick211]

@mphilbrick211 Is this PR still in a product review? or is it ready for merge?

@awais-ansari We haven't heard back from Product - so please hold off on merging.

CC: @jmakowski1123

[jmakowski1123]

This is approved.

[awais-ansari]

@vladislavkeblysh Please update this PR.

[vladislavkeblysh]

@awais-ansari Done, thank you

[mphilbrick211]

All set to merge, @awais-ansari!

[awais-ansari]

@vladislavkeblysh can you please update this PR one more time?

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