kalix-systems/herald

UI bugs

Closed this issue · 0 comments

kaavyajayram commented

Like #71 but for problems that are bugs.

General

  • All text in the UI should be selectable by default, with the exception of text in interactive components. (should be selectable: display names in chat bubbles, headers, settings pane text. shouldn't be selectable: text in clickable PlatonicRectangles, text in buttons, placeholder text in text fields.)
  • "Note to self" conversation not always titled "note to self" on mobile (global search, headers)--should fix by renaming convo "note to self" in rust instead (in a way that can localize)

Chat view

  • Having a conversation open and doing anything in the sidebar should still keep focus in the chat pane so one can still send messages and/or interact with other features of the chat pane, or return focus to the chat pane when one clicks inside it. (Why are we destroying the conversation view and recreating it every time we open group flow and contact flow instead of just overlaying them? This is causing the convo current index to be cleared and hence focus issues)
  • MouseArea in ChatBubbleHover is stealing mouse events from all other MouseAreas in the chat bubble (e.g. download buttons for files)

Group flow

  • In the group flow on mobile and desktop, the file dialogue no longer displays the thumbnail for conversation pictures chosen during the flow, nor sets them correctly (@mobile-bungalow I think you were working with this so you might know how to fix the problem - fileUrl is wrong, but passing it currentFile doesn't seem to fix the problem either)
  • Clicking "create" in new group flow should take you to the chat view for the newly created group

Chat bubbles

  • Figure out sane width behaviour for reply bubbles that are in a message that is not the head of a flurry - this causes the hover buttons to potentially overlap with the bubble (because there is no chat label to push it down), and the behaviour is even worse when there is a download button visible.
  • Reply bubbles with images sometimes overflow height if the text is long. (This might be because the max width for the reply bubbles is not wide enough)
  • In messages that are not the head of a flurry that are a line or less long, the hover buttons should hide the read receipts, not intersect them. (This could be solved with a ternary about the height of the bubble - using the line count would be cleaner though so maybe worth trying to figure out a way to do that before using height)
  • Currently reply bubbles aren't displaying the time info in the correct place (should be on the far right, should also include time till disappearing)

Non-bubble chat view stuff

  • More info view goes away when user interacts with the sidebar; it should stay until X button is clicked
  • More info for Note to Self conversations has empty recipient list, should have the logged-in user as the recipient

Gallery viewer

  • Pinch motions do not work on linux
  • Zooming in and out does not consistently centre the image - it jumps to the side

Image cropping

  • Currently, image does not display at the correct size within the window, and the square used to clip the image is allowed to clip outside of the image (this should not be the case)
  • Area outside square clip over image should be lower opacity/dimmed, possibly a shader required