Fix module spacing when Learning Mode is not enabled

[renatho]

Proposed Changes

• It fixes module spacing when Learning Mode is not enabled.

It didn't happen with Learning Mode enabled because of this style:

sensei/assets/css/3rd-party/themes/course/learning-mode.scss

Line 213 in 96012db

padding-top: 0;

I noticed it happening only with Course theme, but I decided to fix it directly in Sensei core because if it happened for other themes, it would also be fixed. Does it make sense?

Testing Instructions

1. Create a course with at least a module.
2. Deactivate the Learning Mode for this course.
3. Check that the styles look good in the frontend.

Screenshots

Before:

After:

Pre-Merge Checklist

• PR title and description contain sufficient detail and accurately describe the changes
• Acceptance criteria is met
• Decisions are publicly documented
• Adheres to coding standards (PHP, JavaScript, CSS, HTML)
• All strings are translatable (without concatenation, handles plurals)
• Follows our naming conventions (P6rkRX-4oA-p2)
• Hooks (p6rkRX-1uS-p2) and functions are documented
• New UIs are responsive and use a mobile-first approach
• New UIs match the designs
• Different user privileges (admin, teacher, subscriber) are tested as appropriate
• Code is tested on the minimum supported PHP and WordPress versions
• User interface changes have been tested on the latest versions of Chrome, Firefox and Safari
• "Needs Documentation" label is added if this change requires updates to documentation
• Known issues are created as new GitHub issues

[renatho]

This PR only adds this padding. The rest of the changes are just sorting the properties to follow the Sensei standard.

[codecov]

Codecov Report

Merging #7272 (8e68ffa) into trunk (61fefe5) will not change coverage.
Report is 15 commits behind head on trunk.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              trunk    #7272   +/-   ##
=========================================
  Coverage     50.59%   50.59%           
  Complexity    10975    10975           
=========================================
  Files           608      608           
  Lines         46286    46286           
  Branches        402      402           
=========================================
  Hits          23417    23417           
  Misses        22542    22542           
  Partials        327      327           

Files	Coverage Δ
...des/cli/class-sensei-validate-progress-command.php	0.00% <ø> (ø)
...igration/validations/class-progress-validation.php	85.34% <ø> (ø)
...l/migration/validations/class-validation-error.php	100.00% <ø> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96012db...8e68ffa. Read the comment docs.

[m1r0]

I decided to fix it directly in Sensei core because if it happened for other themes, it would also be fixed.

Makes sense to me. 👍 Following the idea of fixing it for other themes, what do you think of using padding: 0 to cover all cases? This would also fix themes that have a padding-bottom for example.

[renatho]

Makes sense to me. 👍 Following the idea of fixing it for other themes, what do you think of using padding: 0 to cover all cases? This would also fix themes that have a padding-bottom for example.

Good idea! Updated here: 8e68ffa

[m1r0]

Looks good. 👍