richie icon indicating copy to clipboard operation
richie copied to clipboard

✨(frontend) compact syllabus for self-paced

Open JoaoGarcao opened this issue 1 year ago • 2 comments

  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

JoaoGarcao avatar Jun 07 '24 13:06 JoaoGarcao

@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 avatar Jun 07 '24 14:06 igobranco

@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).

igobranco avatar Jul 01 '24 10:07 igobranco