Skip to content

Bugfix for dashboard list items

Chris Oelmueller requested to merge bugfix/dashboard-list-items into master

What does this MR do?

Close #455 (closed) : fix dashboard display of markdown list items, and refactor some CSS in the area into SCSS to make that easier to understand.

How confident are you it won't break things if deployed?

90%. It's possible I missed something when refactoring the CSS into SCSS, but I checked most cases and they should still match.

Links to related issues

Slack discussion: https://yunity.slack.com/archives/C74JC2V26/p1588232056243300

How to test

  1. Write dashboard-visible posts (e.g. forum replies) starting with any number followed by a dot (e.g. 2. Sep) or a bullet/dash followed by space
  2. View the resulting post snippet on dashboard

Screenshots (if applicable)

Before:
image

After:
image

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Chris Oelmueller

Merge request reports