-
Notifications
You must be signed in to change notification settings - Fork 76
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
feat: add sticker loading UI #955
Conversation
Closes: #586 Previously, loading sticker packs and stickers would show a lot of blank and non-interactive content. Now, stickers and sticker packs have grey circles to indicate loading. Additionally, the sticker market button (+) shows a loading indicator until the sticker packs are loaded.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool
@@ -260,9 +268,17 @@ QtObject: | |||
write = setActiveChannelByIndex | |||
notify = activeChannelChanged | |||
|
|||
proc getNumInstalledStickerPacks(self: ChatsView): QVariant {.slot.} = | |||
newQVariant(self.status.stickers.installedStickerPacks.len) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could just return this as an int
no?
we shouldn't merge this PR for now due to GC error |
after a chat with @michaelsbradleyjr and @richard-ramos this is ok to merge |
The sticker loading UI was removed when the StatusChatInput and friends were added in. This PR re-adds the sticker loading UI introduced in PR# 955 (#955).
The sticker loading UI was removed when the StatusChatInput and friends were added in. This PR re-adds the sticker loading UI introduced in PR# 955 (#955).
Closes: #586
Previously, loading sticker packs and stickers would show a lot of blank and non-interactive content.
Now, stickers and sticker packs have grey circles to indicate loading. Additionally, the sticker market button (+) shows a loading indicator until the sticker packs are loaded.
Note
If a recent sticker is clicked while the sticker packs are still loading, the app crashes due to some kind of conflict in threads. I spent some time trying to figure this out, but could not get anywhere. Maybe @richard-ramos has an idea?