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.

Sign up for GitHub

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

Jump to bottom

Improve reply rendering #3703

Open
2 tasks
manuroe opened this issue Jul 19, 2021 · 4 comments
Open
2 tasks

Improve reply rendering #3703

manuroe opened this issue Jul 19, 2021 · 4 comments
Labels
A-Timeline X-Needs-Design May require input from the design team Z-parity-with-web Z-WTF WTF moment: High Impact, Low Effort

Comments

@manuroe
Copy link
Member

manuroe commented Jul 19, 2021

From element-ios created by manuroe: element-hq/element-ios#4574

To follow Element-web changes that happened in matrix-org/matrix-react-sdk#3553.
Those changes include new colored bars but also more things that need to be listed:

  • Colored bars
  • TBD...
@manuroe manuroe mentioned this issue Jul 19, 2021
Open
2 tasks
@tulir
Copy link
Contributor

tulir commented Jul 19, 2021

Same as #174? Preferably actually implement reply rendering (strip reply fallback, fetch event from server and render it properly), don't just change the color of the blockquote in reply fallbacks.

@kittykat kittykat added X-Needs-Design May require input from the design team and removed needs-design labels Aug 31, 2021
@daniellekirkwood daniellekirkwood mentioned this issue Jan 6, 2022
Closed
@daniellekirkwood daniellekirkwood added the A-Message-Bubbles Message Bubbles appearance in the Timeline label Jan 10, 2022
@daniellekirkwood
Copy link
Contributor

Tagging Message Bubbles team as we will implement this as a part of the Bubbles work

@daniellekirkwood daniellekirkwood mentioned this issue Jan 11, 2022
Closed
@daniellekirkwood daniellekirkwood added the Z-WTF WTF moment: High Impact, Low Effort label Jan 11, 2022
@daniellekirkwood
Copy link
Contributor

Assigning to @amshakal to provide link to figma for Replies :)

@daniellekirkwood
Copy link
Contributor

Removing the Bubbles label as this issue is no longer tied directly to the bubbles project.
When we take on this work we should consider the Bubbles layout in the app.

@daniellekirkwood daniellekirkwood removed the A-Message-Bubbles Message Bubbles appearance in the Timeline label Mar 8, 2022
@daniellekirkwood daniellekirkwood removed their assignment Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Timeline X-Needs-Design May require input from the design team Z-parity-with-web Z-WTF WTF moment: High Impact, Low Effort
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@kittykat @tulir @manuroe @ganfra @amshakal @daniellekirkwood