✨(plugin) add notifications to courses

[sandroscosta]

Add a new plugin to create warning or information notifications.
This plugin is only available on course detail pages.

Closes #2238

[jbpenrath]

Hey Sandro, look's interesting. I'll have few suggestion to improve your contribution.

First, there is already an existing banner component and I think you could be use it here.

richie/src/richie/apps/courses/templates/courses/cms/fragment_error_detail_template_banner.html

Lines 1 to 14 in fcff6bc

	{% load i18n %}
	{% spaceless %}
	<div class="banner banner--error banner--rounded" role="alert">
	<svg class="banner__icon" aria-hidden="true"><use href="#icon-cross" /></svg>
	<p class="banner__message">
	{% blocktrans %}
	A {{model}} object is missing on this {{model}} page. Please select another page template.
	<br />
	If what you need is a {{model}} page, you need to create it via the wizard and choose "New {{model}} page".
	{% endblocktrans %}
	</p>
	</div>
	{% endspaceless %}

richie/src/frontend/scss/objects/_banner.scss

Line 1 in 37e7995

// Banner

Then, I wonder if this the right place to put the notification.
What do you think to create a static placeholder (like footer) in the global page template ?

[sandroscosta]

Do you think it makes sense for us to meet in the middle? This banner is more "informative" than anything else. The one you show in the images seems to be related more to errors. When I created this, I focused on the course. The idea is to pass messages to the students inside the course page, hence why I chose to put it where it is.

I can refactor with my idea in mind. What do you think? Would a mix of both make sense?

[sandroscosta]

What do you think to create a static placeholder (like footer) in the global page template ?

I don't dislike the idea at all. I just want to restrict this kind of notifications to the courses and programs, for now.

[jbpenrath]

What do you think to create a static placeholder (like footer) in the global page template ?

I don't dislike the idea at all. I just want to restrict this kind of notifications to the courses and programs, for now.

Yup sorry, I realized that the example of the footer placeholder was not a good one.
My suggestion was just to add the notification banner above the course content but of course this placeholder should be unique for each page.

[sandroscosta]

My suggestion was just to add the notification banner above the course content but of course this placeholder should be unique for each page.

Make sense. I'll check and work on it. I might create some additional styles to have different levels of alert states.