Skip to content

Make mail recipients list foldable

Alex requested to merge 65-foldable-mail-recipients into master

Closes #65 (closed)

What does this MR do?

Adds a button that folds the list of mail recipients, so that mails with many recipients are still readable. The button is only added if the list is too long (>100 characters) and is set to folded as default.

I'm not so sure whether the icon is good for this. It currently uses fa-sort-down and fa-sort-up.

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

Quite sure, shouldn't affect other things.

How to test

Steps a reviewer can take to verify that this MR does what it says it does e.g.

  1. Checkout branch locally
  2. Login as someone with a mailbox
  3. Send a mail to many recipients (see e.g. the list in the issue)
  4. Open the mail in the sent folder

Screenshots (if applicable)

unfolded folded

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 Alex

Merge request reports