Skip to content

WIP: Redesign chatbox and messages page

Christian Walgenbach requested to merge set-important-for-my-message-link-in-chat into master

What does this MR do?

At the moment, you can not easily recognize links. After consultation with Alex from the grafik ag, I changed the colors of the chatbox and messages page.

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

(be honest!)

Links to related issues

https://yunity.slack.com/archives/C1T1ZM9A4/p1574362939261200

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 amb (bot).
  3. Write a chat message to user two with a link.
  4. Logout and login as user two and check the message (chatbox and message box) for legibility.
  5. Write a message to amb with a link.
  6. Logout and login as amb and check the message (chatbox and message box) for legibility.

Screenshots (if applicable)

Suggestion:
chatbox3

message-page2

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 Christian Walgenbach

Merge request reports