openfun / richie

:pencil: An opensource CMS to build education portals
https://richie.education
MIT License
260 stars 84 forks source link

✨(frontend) compact syllabus for self-paced #2428

Closed JoaoGarcao closed 2 months ago

JoaoGarcao commented 3 months ago
  1. If all runs of an opened course have the same languages, we hide the languages on the course run card, because those languages are shown after the course description.
  2. On self-paced courses with single opened run, display a simplified version of the syllabus.

fix #1969

Change the message of the enrollment period for a course with single opened run when it is self-paced, and hide the languages of a course when every run of that course has the same languages as all the others.

Opened self-paced course:

This PR shows a compacted version of the information. Before:

1

After:

2

Opened course with the same languages as all the other runs of that course

As all runs for the course as the same languages, showing the same exact information multiple times is unnecessary, so this PR removes that languages on each run syllabus. This behavior is the same for both self-paced and instructor-paced courses. Before:

3

After:

4

Instructor course with different languages, stays the same

If a course isn't self-paced (instructor pace) and the languages of all runs of that course aren't equal, then the course run card stays the same as it was:

5

Compacted version for self-paced & same languages for all runs

If an opened course run is self-paced and the languages of all runs of that course are equal, then the course run card will be as displayed bellow. This is the normal/majority of the courses in NAU.

5

Compacted version for self-paced, same languages for all runs & no course end date

if we have the case described above, but also there is no course end date, then the course run card will be:

1

FYI @igobranco @sandroscosta

igobranco commented 3 months ago

@JoaoGarcao please fix the lint's.

make lint-back
make lint-front

On the last image scenario, Compacted version for self-paced & same languages for all runs, please remove the Enrollment. This means that the header with the Enrollment as text shouldn't be displayed on the compacted version.

igobranco commented 3 months ago

@jbpenrath - I haven't present to you @JoaoGarcao , but he is a student that have been doing a curricular internship on NAU. I've already reviewed this PR with him, but I think you or someone from FUN should review this PR, before we merge it. This PR has impact on almost 100% of NAU course pages, but on you side as lower impact ( < 50% opened are self paced).