Skip to content

Add loading indicator#496

Closed
yuraiz wants to merge 1 commit into
paper-plane-developers:mainfrom
yuraiz:loading-indicator
Closed

Add loading indicator#496
yuraiz wants to merge 1 commit into
paper-plane-developers:mainfrom
yuraiz:loading-indicator

Conversation

@yuraiz
Copy link
Copy Markdown
Contributor

@yuraiz yuraiz commented Mar 7, 2023

Adds loading indicator to MessageDocument, I plan to use the same indicator for other media messages

Indicator drawn with cairo because I want arc to have rounded end points.
Rotation done inside widget instead of css because otherwise it rotates texture and it doesn't look good

@yuraiz yuraiz changed the title message-document: Add thumbnails Add loading indicator Mar 8, 2023
@yuraiz yuraiz force-pushed the loading-indicator branch 3 times, most recently from 4898d24 to 6e8965e Compare March 12, 2023 18:06
@yuraiz yuraiz marked this pull request as ready for review March 12, 2023 18:08
@melix99 melix99 added this to the Telegrand 0.1 milestone Mar 22, 2023
Comment thread src/components/loading_indicator.rs Outdated
@yuraiz yuraiz force-pushed the loading-indicator branch from 6e8965e to 8286835 Compare March 23, 2023 08:46
@yuraiz yuraiz requested a review from melix99 March 23, 2023 08:48
@yuraiz yuraiz force-pushed the loading-indicator branch from 8286835 to 74148cd Compare March 23, 2023 08:54
Comment thread src/components/loading_indicator.rs Outdated
Comment thread src/components/loading_indicator.rs
Comment thread src/components/loading_indicator.rs Outdated
Comment thread src/components/loading_indicator.rs Outdated
Comment thread src/components/loading_indicator.rs Outdated
@yuraiz yuraiz force-pushed the loading-indicator branch from 74148cd to f47fc26 Compare March 24, 2023 18:56
@yuraiz yuraiz requested a review from melix99 March 24, 2023 18:56
@marhkb
Copy link
Copy Markdown
Collaborator

marhkb commented Sep 30, 2023

Closed in favor of #590

@marhkb marhkb closed this Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants