update notification styling

[indirectlylit]

Summary

before	after

Reviewer guidance

make sense?

any regressions?

---

Contributor Checklist

PR process:

• PR has the correct target branch and milestone
• PR has 'needs review' or 'work-in-progress' label
• If PR is ready for review, a reviewer has been added. (Don't use 'Assignees')
• If this is an important user-facing change, PR or related issue has a 'changelog' label
• If this includes an internal dependency change, a link to the diff is provided

Testing:

• Contributor has fully tested the PR manually
• If there are any front-end changes, before/after screenshots are included
• Critical user journeys are covered by Gherkin stories
• Critical and brittle code paths are covered by unit tests

Reviewer Checklist

• Automated test coverage is satisfactory
• PR is fully functional
• PR has been tested for accessibility regressions
• External dependency files were updated if necessary (yarn and pip)
• Documentation is updated
• Contributor is in AUTHORS.md

[radinamatic]

I actually liked the old version of the notification, it was impossible to miss.

This new one is better styled, true, but so easy to overlook 😕

[jonboiser]

This test needs to be fixed here: https://travis-ci.org/github/learningequality/kolibri/jobs/711165802#L8356

It might simply just involve flipping the indices of the paragraphs to get the right text.

[indirectlylit]

this better?

[radinamatic]

Yes, this does look more prominent 👍

We could discuss adding a different background color for that block (like for old notifications), to make it pop out even more, but this looks OK for now.

[indirectlylit]

fixed linting, force-merging