fix: large text for rating

[johnvente]

Description

This PR adds a small fix for when the rating text is too long for the options and increases the 'Number of ratings' to provide more space for feedback in the instructor tab.

BEFORE - RATING

NOW - RATING

BEFORE - INSTRUCTOR TAB

NOW -INSTRUCTOR TAB

How to test it

1. Check this branch inside the XBlock

 git fetch origin pull/70/head:jv/fix-feedback-ui-problems
 git checkoutjv/fix-feedback-ui-problems

2. In your environment add the following setting as doc

name: feedback-xblock-settings
version: 0.1.0
patches:
  openedx-common-settings: |
    FEATURES["ENABLE_FEEDBACK_INSTRUCTOR_VIEW"] = True
    OPEN_EDX_FILTERS_CONFIG = {
      "org.openedx.learning.instructor.dashboard.render.started.v1": {
        "fail_silently": False,
        "pipeline": [
          "feedback.extensions.filters.AddFeedbackTab",
        ]
      },
    }

3. From Studio, in your course > Advanced Settings > Advanced Module List > add "feedback" in the list.

4. In a new unit add a feedback component

5. Edit the component and put a long text for each Likert option save and check if the rating are in row

[openedx-webhooks]

Thanks for the pull request, @johnvente! 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.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.47%. Comparing base (ae9ef41) to head (4126ab6).

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   92.47%   92.47%           
=======================================
  Files           5        5           
  Lines         226      226           
=======================================
  Hits          209      209           
  Misses         17       17           

Flag	Coverage Δ
unittests	92.47% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[johnvente]

Hi there @ziafazal! I put you here as reviewer due to you have reviewed some pr's from edunext, could you take a look over here please? Thanks a lot!

[ziafazal]

@johnvente Thanks for creating PR with improved styling. I have couple of suggestions if you can look into those it would be great.

[johnvente]

Hi @ziafazal I addressed one of your suggestion please let me know if is okay

[ziafazal]

👍

[mariajgrimaldi]

@feanil: could you help us merge this PR? Thanks!

[mariajgrimaldi]

Hi @farhan! Could you help us review/merge this? Thanks!

[johnvente]

I won't be able to review this in a week so I will check any changes 1 April

[farhan]

@johnvente
Thanks for the work. Please rebase and resolve conflicts

[mariajgrimaldi]

@farhan: Jhon won't be available until April 1st, so I rebased this instead. Could you check again? Thanks!

[openedx-webhooks]

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