Skip to content
Snippets Groups Projects

Chat improvements

Merged Christian Nienhaus requested to merge nini1988-chat-improvements into master

What does this MR do?

  • Increase popup chat size.
  • Limit message textarea max height
  • Prevent scrolling page when inside of popup chat window
  • Hide one check for send message. Confuses people when no second check will be displayed (backend does not support distributed information)
  • Display two rows for titel and last message in conversation list.
  • Increase text space in popup chats

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

confident

Links to related issues

How to test

Screenshots (if applicable)

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • set a "for:" label to indicate who will be affected by this change
  • added to the next milestone (see https://gitlab.com/foodsharing-dev/foodsharing/-/milestones, unless it has a "for:Dev" label)
  • added an entry to CHANGELOG.md
  • added a short text that can be used in the release notes
  • Once your MR has been merged, you are responsible to create a testing issue in the Beta Testing forum: https://foodsharing.de/?page=bezirk&bid=734&sub=forum. Please change the MRs label to "state:Beta testing".
    • Consider writing a detailed description in German.
    • Describe in a few sentences, what should be tested from a user perspective.
    • Also mention different settings (e.g. different browsers, roles, ...) how this change can be tested.
    • Be aware, that also non technical people should understand.

Release notes text in German

Edited by Christian Nienhaus

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • 7dc72dd7 - Apply 2 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Christian Nienhaus resolved all threads

    resolved all threads

  • Christian Nienhaus marked this merge request as ready

    marked this merge request as ready

  • added 1 commit

    Compare with previous version

  • Christian Nienhaus resolved all threads

    resolved all threads

  • Martin approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading