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

Room screen: Timeline UI #34

Closed
manuroe opened this issue May 4, 2022 · 0 comments · Fixed by #91
Closed

Room screen: Timeline UI #34

manuroe opened this issue May 4, 2022 · 0 comments · Fixed by #91
Assignees
Labels
A-Timeline X-Needs-Design Z-BBQ-Alpha Required for ElementX - BBQ - Alpha

Comments

@manuroe
Copy link
Member

manuroe commented May 4, 2022

We only know that ElementX will use message bubbles.

Can we reuse the design recently implemented on EI?

@manuroe manuroe added Z-BBQ-Alpha Required for ElementX - BBQ - Alpha X-Needs-Design A-Timeline labels May 4, 2022
@ismailgulek ismailgulek self-assigned this Jun 21, 2022
ismailgulek added a commit that referenced this issue Jun 22, 2022
ismailgulek added a commit that referenced this issue Jun 22, 2022
ismailgulek added a commit that referenced this issue Jun 22, 2022
ismailgulek added a commit that referenced this issue Jun 22, 2022
ismailgulek added a commit that referenced this issue Jun 22, 2022
ismailgulek added a commit that referenced this issue Jun 23, 2022
ismailgulek added a commit that referenced this issue Jun 23, 2022
ismailgulek added a commit that referenced this issue Jun 23, 2022
ismailgulek added a commit that referenced this issue Jun 23, 2022
* #34 Create `TimelineItemStylerView`

* #34 Add styler view into different type of cells

* #34 Use placeholder avatar image on room list

* #34 Add `isOutgoing` param to event based timeline item

* #34 Compute `isOutgoing` for timeline items

* #34 Update sender info view in timeline

* #34 Update mock timeline items

* #34 Rename `EventBasedTimelineView` to `EventBasedTimelineSenderView`

* #34 Change padding on timeline cells

* #34 Create `TimelineItemStylerView` to move content into a bubble if needed

* #34 Use styler view in all of the timeline item views

* #34 Make timestamp more readable on images

* #34 Little layout tweaks

* #34 Add changelog

* #34 Fix code smells

* #34 Set text colors on timeline items

* #34 Fix background color of the timeline

* #34 Fix PR remarks

* #34 Set background colors explicitly on remaining screens

* #34 Reduce min bubble width and make it a scaled metric

* #34 Refactor `PlaceholderAvatarImage` to accept a text only

* #34 Fix code smell

* #34 Fix further comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline X-Needs-Design Z-BBQ-Alpha Required for ElementX - BBQ - Alpha
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants