Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create chat view #29

Open
benjaminpreiss opened this issue Mar 19, 2025 · 5 comments · Fixed by #30
Open

Create chat view #29

benjaminpreiss opened this issue Mar 19, 2025 · 5 comments · Fixed by #30
Assignees
Labels

Comments

@benjaminpreiss
Copy link
Collaborator

benjaminpreiss commented Mar 19, 2025

Create first version of chat view, detailed by the screenshots below:

This is the chat view for now:
Image

This is the styling for the quote elements used:
Image

Logics:

  • Clicking on a post doesn't navigate, but expand the post
  • For now, we enable switching by clicking on the top right sort button (so chat view is a sort option)
@benjaminpreiss benjaminpreiss self-assigned this Mar 19, 2025
@benjaminpreiss benjaminpreiss linked a pull request Mar 19, 2025 that will close this issue
@benjaminpreiss
Copy link
Collaborator Author

Discussions with Erik:

  1. Hold-click on a post in chat view opens context menu
  2. Leave meatballs (3 dots) to activate context menu too (grandma friendly)
  3. Implement auto-reply feature
  4. Make images as full-screen as possible, without destroying right/left alignment
  5. Click doesn't navigate in chat-view, but opens the context of this msg (reply stack + replies)

These are the wireframes that have to be implemented:

Image

@benjaminpreiss
Copy link
Collaborator Author

benjaminpreiss commented Mar 20, 2025

Newest design to implement:

Image

@benjaminpreiss
Copy link
Collaborator Author

benjaminpreiss commented Mar 20, 2025

Implementation procedure:

  • Separate post contents into framed boxes
  • Make images not full-screen in mobile view
  • Show all nested posts of current breadcrumb (not only immediate children)
  • Add lines on replies
  • that the app pane should show up if you want to add more stuff to the view while it already full of stuff. (*)
  • make full-screen button always visible, if you added at least one app or typed one letter

Example for (*):

Image

@marcus-pousette
Copy link
Member

Image

Looks like the chat view changes also somehow has affected how text is rendered in the feed/thread view. I think the behaviour before that there where no borders there

@benjaminpreiss
Copy link
Collaborator Author

  • reply button to reply
  • writing a new message automatically replies to previous one
  • either switch of auto-reply with an x icon or by clicking on another message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants