Properly add/remove sticker sets (almost ;))

qml/components/StickerPicker.qml

@@ -25,22 +25,32 @@ Item {
     id: stickerPickerOverlayItem
     anchors.fill: parent
 
-    property var recentStickers: stickerManager.getRecentStickers()
+    property var recentStickers
-    property var installedStickerSets: stickerManager.getInstalledStickerSets()
+    property var installedStickerSets
-    property bool pickerLoaded: false
+    property int setInRemoval: -1
+
+    Component.onCompleted: {
+        recentStickers = stickerManager.getRecentStickers();
+        installedStickerSets = stickerManager.getInstalledStickerSets();
+    }
+
+    Connections {
+        target: tdLibWrapper
+        onOkReceived: {
+            if (request === "removeStickerSet") {
+                appNotification.show(qsTr("Sticker set successfully removed!"));
+                tdLibWrapper.getInstalledStickerSets();
+                console.log("Set in removal: " + setInRemoval);
+                if (setInRemoval > -1) {
+                    installedStickerSets.splice(setInRemoval, 1);
+                    setInRemoval = -1;
+                }
+            }
+        }
+    }
 
     signal stickerPicked(var stickerId)
 
-    Timer {
-        id: stickerPickerLoadedTimer
-        interval: 100
-        running: true
-        repeat: false
-        onTriggered: {
-            stickerPickerOverlayItem.pickerLoaded = true;
-        }
-    }
-
     Rectangle {
         id: stickerPickerOverlayBackground
         anchors.fill: parent
@@ -54,10 +64,6 @@ Item {
         anchors.fill: parent
         anchors.margins: Theme.paddingMedium
 
-        opacity: stickerPickerOverlayItem.pickerLoaded ? 1 : 0
-        Behavior on opacity { NumberAnimation {} }
-        visible: stickerPickerOverlayItem.pickerLoaded
-
         contentHeight: stickerPickerColumn.height
         clip: true
 
@@ -66,7 +72,7 @@ Item {
             spacing: Theme.paddingMedium
             width: stickerPickerFlickable.width
             Label {
-                font.pixelSize: Theme.fontSizeMedium
+                font.pixelSize: Theme.fontSizeLarge
                 font.bold: true
                 width: parent.width
                 maximumLineCount: 1
@@ -90,15 +96,16 @@ Item {
                     width: recentStickersGridView.cellWidth
                     height: recentStickersGridView.cellHeight
 
-                    Image {
+                    TDLibThumbnail {
-                        source: modelData.thumbnail.file.local.path
+                        thumbnail: modelData.thumbnail
                         anchors.fill: parent
-                        asynchronous: true
-                        onStatusChanged: {
-                            if (status === Image.Ready) {
-                                stickerPickerLoadedTimer.restart();
-                            }
                     }
+
+                    Label {
+                        font.pixelSize: Theme.fontSizeSmall
+                        anchors.right: parent.right
+                        anchors.bottom: parent.bottom
+                        text: Emoji.emojify(modelData.emoji, font.pixelSize)
                     }
 
                     MouseArea {
@@ -118,15 +125,34 @@ Item {
                 Column {
                     spacing: Theme.paddingMedium
                     width: parent.width
-                    Label {
+                    Row {
-                        font.pixelSize: Theme.fontSizeMedium
-                        font.bold: true
                         width: parent.width
+                        height: Math.max(removeSetButton.height, setTitleText.height) + ( 2 * Theme.paddingSmall )
+                        Label {
+                            id: setTitleText
+                            font.pixelSize: Theme.fontSizeLarge
+                            font.bold: true
+                            width: parent.width - removeSetButton.width
+                            anchors.verticalCenter: parent.verticalCenter
                             maximumLineCount: 1
                             truncationMode: TruncationMode.Fade
                             text: modelData.title
                         }
 
+                        IconButton {
+                            id: removeSetButton
+                            icon.source: "image://theme/icon-m-remove"
+                            anchors.verticalCenter: parent.verticalCenter
+                            onClicked: {
+                                var stickerSetId = modelData.id;
+                                setInRemoval = index;
+                                Remorse.popupAction(chatPage, qsTr("Removing sticker set"), function() {
+                                    tdLibWrapper.changeStickerSet(stickerSetId, false);
+                                });
+                            }
+                        }
+                    }
+
                     SilicaGridView {
                         id: installedStickerSetGridView
                         width: parent.width
@@ -142,25 +168,16 @@ Item {
                             width: installedStickerSetGridView.cellWidth
                             height: installedStickerSetGridView.cellHeight
 
-                            Image {
+                            TDLibThumbnail {
-                                id: singleStickerImage
+                                thumbnail: modelData.thumbnail
-                                source: modelData.thumbnail.file.local.is_downloading_completed ? modelData.thumbnail.file.local.path : ""
                                 anchors.fill: parent
-                                visible: modelData.thumbnail.file.local.is_downloading_completed
-                                asynchronous: true
-                                onStatusChanged: {
-                                    if (status === Image.Ready) {
-                                        stickerPickerLoadedTimer.restart();
-                                    }
-                                }
                             }
+
                             Label {
-                                font.pixelSize: Theme.fontSizeHuge
+                                font.pixelSize: Theme.fontSizeSmall
-                                anchors.fill: parent
+                                anchors.right: parent.right
-                                maximumLineCount: 1
+                                anchors.bottom: parent.bottom
-                                truncationMode: TruncationMode.Fade
                                 text: Emoji.emojify(modelData.emoji, font.pixelSize)
-                                visible: !modelData.thumbnail.file.local.is_downloading_completed
                             }
 
                             MouseArea {
@@ -177,27 +194,5 @@ Item {
         }
     }
 
-    Column {
-        anchors.centerIn: parent
-        width: parent.width
-        spacing: Theme.paddingMedium
-
-        opacity: stickerPickerOverlayItem.pickerLoaded ? 0 : 1
-        Behavior on opacity { NumberAnimation {} }
-        visible: !stickerPickerOverlayItem.pickerLoaded
-
-        InfoLabel {
-            id: loadingLabel
-            text: qsTr("Loading stickers...")
-        }
-
-        BusyIndicator {
-            id: loadingBusyIndicator
-            anchors.horizontalCenter: parent.horizontalCenter
-            running: !stickerPickerOverlayItem.pickerLoaded
-            size: BusyIndicatorSize.Large
-        }
-    }
-
 }
 

qml/components/StickerSetOverlay.qml

@@ -32,7 +32,6 @@ Flickable {
 
     property string stickerSetId;
     property var stickerSet;
-    property bool setLoaded: false;
     signal requestClose;
 
     Component.onCompleted: {
@@ -50,15 +49,17 @@ Flickable {
                 stickerSetOverlayFlickable.stickerSet = stickerSet;
             }
         }
+        onOkReceived: {
+            if (request === "installStickerSet") {
+                appNotification.show(qsTr("Sticker set successfully installed!"));
+                installSetButton.visible = false;
+                tdLibWrapper.getInstalledStickerSets();
+            }
+            if (request === "removeStickerSet") {
+                appNotification.show(qsTr("Sticker set successfully removed!"));
+                installSetButton.visible = true;
+                tdLibWrapper.getInstalledStickerSets();
             }
-
-    Timer {
-        id: stickerSetLoadedTimer
-        interval: 100
-        running: true
-        repeat: false
-        onTriggered: {
-            stickerSetOverlayFlickable.setLoaded = true;
         }
     }
 
@@ -90,7 +91,7 @@ Flickable {
             Label {
                 id: overlayStickerTitleText
 
-                width: parent.width - ( installSetButton.visible ? installSetButton.width : 0 ) - closeSetButton.width
+                width: parent.width - installSetButton.width - closeSetButton.width
                 text: stickerSet.title
                 font.pixelSize: Theme.fontSizeExtraLarge
                 font.weight: Font.ExtraBold
@@ -104,12 +105,22 @@ Flickable {
                 id: installSetButton
                 icon.source: "image://theme/icon-m-add"
                 anchors.verticalCenter: parent.verticalCenter
-                visible: !stickerSet.is_installed
+                visible: !stickerManager.isStickerSetInstalled(stickerSet.id)
                 onClicked: {
                     tdLibWrapper.changeStickerSet(stickerSet.id, true);
                 }
             }
 
+            IconButton {
+                id: removeSetButton
+                icon.source: "image://theme/icon-m-remove"
+                anchors.verticalCenter: parent.verticalCenter
+                visible: !installSetButton.visible
+                onClicked: {
+                    tdLibWrapper.changeStickerSet(stickerSet.id, false);
+                }
+            }
+
             IconButton {
                 id: closeSetButton
                 icon.source: "image://theme/icon-m-clear"
@@ -139,25 +150,17 @@ Flickable {
                 width: stickerSetGridView.cellWidth - Theme.paddingSmall
                 height: stickerSetGridView.cellHeight - Theme.paddingSmall
 
-                Image {
+                TDLibThumbnail {
-                    id: singleStickerImage
+                    id: singleStickerThumbnail
-                    source: modelData.thumbnail.file.local.is_downloading_completed ? modelData.thumbnail.file.local.path : ""
+                    thumbnail: modelData.thumbnail
                     anchors.fill: parent
-                    visible: modelData.thumbnail.file.local.is_downloading_completed
-                    asynchronous: true
-                    onStatusChanged: {
-                        if (status === Image.Ready) {
-                            stickerSetLoadedTimer.restart();
-                        }
-                    }
                 }
+
                 Label {
-                    font.pixelSize: Theme.fontSizeHuge
+                    font.pixelSize: Theme.fontSizeSmall
-                    anchors.fill: parent
+                    anchors.right: parent.right
-                    maximumLineCount: 1
+                    anchors.bottom: parent.bottom
-                    truncationMode: TruncationMode.Fade
                     text: Emoji.emojify(modelData.emoji, font.pixelSize)
-                    visible: !modelData.thumbnail.file.local.is_downloading_completed
                 }
 
             }

qml/pages/ChatPage.qml

@@ -699,7 +699,7 @@ Page {
         contentWidth: width
 
         PullDownMenu {
-            visible: chatInformation.id !== chatPage.myUserId && !stickerPickerLoader.active && !voiceNoteOverlayLoader.active && !messageOverlayLoader.active
+            visible: chatInformation.id !== chatPage.myUserId && !stickerPickerLoader.active && !voiceNoteOverlayLoader.active && !messageOverlayLoader.active && !stickerSetOverlayLoader.active
             MenuItem {
                 id: closeSecretChatMenuItem
                 visible: chatPage.isSecretChat && chatPage.secretChatDetails.state["@type"] !== "secretChatStateClosed"
@@ -958,7 +958,7 @@ Page {
                     id: chatView
 
                     visible: !blurred
-                    property bool blurred: messageOverlayLoader.item || stickerPickerLoader.item || voiceNoteOverlayLoader.item || inlineQuery.hasOverlay
+                    property bool blurred: messageOverlayLoader.item || stickerPickerLoader.item || voiceNoteOverlayLoader.item || inlineQuery.hasOverlay || stickerSetOverlayLoader.item
 
                     anchors.fill: parent
                     opacity: chatPage.loading ? 0 : 1
@@ -1304,6 +1304,9 @@ Page {
                     }
 
                     onActiveChanged: {
+                        if (active) {
+                            attachmentOptionsFlickable.isNeeded = false;
+                        }
                     }
                 }
 
@@ -1773,7 +1776,7 @@ Page {
                         icon.source: "image://theme/icon-m-attach?" +  (attachmentOptionsFlickable.isNeeded ? Theme.highlightColor : Theme.primaryColor)
                         anchors.bottom: parent.bottom
                         anchors.bottomMargin: Theme.paddingSmall
-                        enabled: !attachmentPreviewRow.visible
+                        enabled: !attachmentPreviewRow.visible && !stickerSetOverlayLoader.item
                         visible: !inlineQuery.userNameIsValid
                         onClicked: {
                             if (attachmentOptionsFlickable.isNeeded) {

src/stickermanager.cpp

@@ -60,6 +60,11 @@ bool StickerManager::hasStickerSet(const QString &stickerSetId)
     return this->stickerSets.contains(stickerSetId);
 }
 
+bool StickerManager::isStickerSetInstalled(const QString &stickerSetId)
+{
+    return this->installedStickerSetIds.contains(stickerSetId);
+}
+
 bool StickerManager::needsReload()
 {
     return this->reloadNeeded;
@@ -105,7 +110,15 @@ void StickerManager::handleStickerSetsReceived(const QVariantList &stickerSets)
     QListIterator<QVariant> stickerSetsIterator(stickerSets);
     while (stickerSetsIterator.hasNext()) {
         QVariantMap newStickerSet = stickerSetsIterator.next().toMap();
-        this->stickerSets.insert(newStickerSet.value("id").toString(), newStickerSet);
+        QString newSetId = newStickerSet.value("id").toString();
+        bool isInstalled = newStickerSet.value("is_installed").toBool();
+        if (isInstalled && !this->installedStickerSetIds.contains(newSetId)) {
+            this->installedStickerSetIds.append(newSetId);
+        }
+        if (!isInstalled && this->installedStickerSetIds.contains(newSetId)) {
+            this->installedStickerSetIds.removeAll(newSetId);
+        }
+        this->stickerSets.insert(newSetId, newStickerSet);
     }
 
     this->installedStickerSets.clear();

src/stickermanager.h

@@ -37,6 +37,7 @@ public:
     Q_INVOKABLE QVariantList getInstalledStickerSets();
     Q_INVOKABLE QVariantMap getStickerSet(const QString &stickerSetId);
     Q_INVOKABLE bool hasStickerSet(const QString &stickerSetId);
+    Q_INVOKABLE bool isStickerSetInstalled(const QString &stickerSetId);
     Q_INVOKABLE bool needsReload();
     Q_INVOKABLE void setNeedsReload(const bool &reloadNeeded);
 

src/tdlibwrapper.cpp

@@ -1332,6 +1332,7 @@ void TDLibWrapper::changeStickerSet(const QString &stickerSetId, bool isInstalle
 
     QVariantMap requestObject;
     requestObject.insert(_TYPE, "changeStickerSet");
+    requestObject.insert(_EXTRA, isInstalled ? "installStickerSet" : "removeStickerSet");
     requestObject.insert("set_id", stickerSetId);
     requestObject.insert("is_installed", isInstalled);
 

translations/harbour-fernschreiber-de.ts

@@ -1666,8 +1666,23 @@
         <translation>Kürzlich verwendet</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Lade Sticker...</translation>
+        <translation>Entferne Sticker-Set</translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation>Sticker-Set erfolgreich entfernt!</translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation>Sticker-Set erfolgreich installiert!</translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation>Sticker-Set erfolgreich entfernt!</translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-en.ts

@@ -1668,8 +1668,23 @@ messages</numerusform>
         <translation>Recently used</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Loading stickers...</translation>
+        <translation>Removing sticker set</translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation>Sticker set successfully removed!</translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation>Sticker set successfully installed!</translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation>Sticker set successfully removed!</translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-es.ts

@@ -1666,8 +1666,23 @@
         <translation>Usado recientemente</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Cargando pegatinas...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-fi.ts

@@ -1667,8 +1667,23 @@
         <translation>Viimeksi käytetty</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Ladataan tarroja...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-hu.ts

@@ -1639,7 +1639,22 @@
         <translation type="unfinished"></translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
         <translation type="unfinished"></translation>
     </message>
 </context>

translations/harbour-fernschreiber-it.ts

@@ -1666,8 +1666,23 @@
         <translation>Usati di recente</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Carica sticker...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-pl.ts

@@ -1693,8 +1693,23 @@
         <translation>Ostatnio użyty</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Ładowanie naklejek...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-ru.ts

@@ -1696,8 +1696,23 @@
         <translation>Недавно использованные</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Загрузка стикеров...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-sk.ts

@@ -1693,8 +1693,23 @@
         <translation>Nedávno použité</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Načítanie nálepiek...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-sv.ts

@@ -1666,8 +1666,23 @@
         <translation>Nyligen använt</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Läser in dekaler...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber-zh_CN.ts

@@ -1640,8 +1640,23 @@
         <translation>最近使用</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>正在加载表情贴图…</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>

translations/harbour-fernschreiber.ts

@@ -1666,8 +1666,23 @@
         <translation>Recently used</translation>
     </message>
     <message>
-        <source>Loading stickers...</source>
+        <source>Removing sticker set</source>
-        <translation>Loading stickers...</translation>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+</context>
+<context>
+    <name>StickerSetOverlay</name>
+    <message>
+        <source>Sticker set successfully installed!</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Sticker set successfully removed!</source>
+        <translation type="unfinished"></translation>
     </message>
 </context>
 <context>