Skip to content

Bugfixes for #876 and #722 (UI overflow) and #869 (Forum bell thread title)

Chris Oelmueller requested to merge bugfix/876-722-ui-overflow into master

Closes #876 (closed)
Closes #722 (closed)
Closes #869 (closed)

What does this MR do?

Fix some UI overflow problems, and include thread title in forum bell notifications.

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

The two overflow tickets: Pretty confident :) it's just CSS in any case

The bell notifications? might need backwards compatibility for old bells that didn't store the title

How to test

  • see #876 (closed) for reproduction steps to test event overflow
  • see #722 (closed) for reproduction steps to test notification box overflow
  • generate forum bells to check the thread title is now included

Screenshots

Event page (Before) image image
Event page (After) image image
Notification box (Before)

image

Notification box (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