D27346: emojier: Improve resizing performance

apol abandoned this revision. apol added a comment. Solved as https://commits.kde.org/plasma-desktop/91c446e0c4430289369963bf44e64228add67d18 REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27346 To: apol, #plasma Cc: ngraham, Zren, plasma-devel, Orage, LeGas

D27346: emojier: Improve resizing performance

Zren added a comment. As in `clip: true`? delegate: MouseArea { width: emojiView.cellWidth height: emojiView.cellHeight clip: true Rectangle { border.color: "#ff0"; anchors.fill: parent; border.width: 1; color: "transparent" } QQC2.Label

D27346: emojier: Improve resizing performance

apol added a comment. In D27346#612081 , @ngraham wrote: > What's the status of this? Is it ready? No, it's not. @Zren Another thing we could do is taking such multiglyph renders as a bug on the font. We can clip them so they don'

D27346: emojier: Improve resizing performance

ngraham added a comment. What's the status of this? Is it ready? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27346 To: apol, #plasma Cc: ngraham, Zren, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen

D27346: emojier: Improve resizing performance

Zren added a comment. Not sure. Backwards compatibility? Firefox renders a couple of them fine, while the TaskManager widget doesn't. Could it be the "Noto" font? The facepalm ones not working are 3 glyphs merged together (facepalm+skincolor+sex). It's rendering (facepalm+skincolor) fine

D27346: emojier: Improve resizing performance

apol added a comment. uh right, didn't see that. Actually, do you know what's up with those? we should just be showing them as 1 glyph... REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27346 To: apol, #plasma Cc: Zren, plasma-devel, Orage, LeGast00n, The-Fer

D27346: emojier: Improve resizing performance

Zren added a comment. FixedSize 25pt does help, but "2 glyph emojis" will overlap each other. F8099880: 2020-02-12___11-38-36.png F8099891: screen-2020-02-12_11.34.09.mp4 Rectangle { width: pare

D27346: emojier: Improve resizing performance

apol created this revision. apol added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Don't calculate the most fitting size, adopt a reasonable size and go with it. BUG: 417454 TEST PLAN