fix: Use Qt's High-DPI scaling

Chatterino · Oct 5, 2023 · 2636eeb · 2636eeb
1 parent 7c8caba
commit 2636eeb
Show file tree

Hide file tree

Showing 17 changed files with 161 additions and 190 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -17,6 +17,7 @@
 - Bugfix: Fixed the Quick Switcher (CTRL+K) from sometimes showing up on the wrong window. (#4819)
 - Bugfix: Fixed too much text being copied when copying chat messages. (#4812, #4830, #4839)
 - Bugfix: Fixed empty page being added when showing out of bounds dialog. (#4849)
+- Bugfix: Fixed broken scaling in context menus. (#4868)
 - Dev: Fixed UTF16 encoding of `modes` file for the installer. (#4791)
 - Dev: Temporarily disable High DPI scaling on Qt6 builds on Windows. (#4767)
 - Dev: Tests now run on Ubuntu 22.04 instead of 20.04 to loosen C++ restrictions in tests. (#4774)

diff --git a/resources/qss/settings.qss b/resources/qss/settings.qss
@@ -1,11 +1,11 @@
 * {
-    font-size: <font-size>px;
+    font-size: 14px;
     font-family: "Segoe UI";
 }
 
 QCheckBox::indicator {
-    width: <checkbox-size>px;
-    height: <checkbox-size>px;
+    width: 14px;
+    height: 14px;
 }
 
 chatterino--ComboBox {

diff --git a/src/RunGui.cpp b/src/RunGui.cpp
@@ -75,11 +75,7 @@ namespace {
     void initQt()
     {
         // set up the QApplication flags
-        QApplication::setAttribute(Qt::AA_Use96Dpi, true);
-#ifdef Q_OS_WIN32
-        QApplication::setAttribute(Qt::AA_DisableHighDpiScaling, true);
-#endif
-
+        QApplication::setAttribute(Qt::AA_Use96Dpi);
         QApplication::setStyle(QStyleFactory::create("Fusion"));
 
 #ifndef Q_OS_MAC

diff --git a/src/main.cpp b/src/main.cpp
@@ -24,11 +24,6 @@ using namespace chatterino;
 
 int main(int argc, char **argv)
 {
-    // TODO: This is a temporary fix (see #4552).
-#if defined(Q_OS_WINDOWS) && QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)
-    qputenv("QT_ENABLE_HIGHDPI_SCALING", "0");
-#endif
-
     QApplication a(argc, argv);
 
     QCoreApplication::setApplicationName("chatterino");

diff --git a/src/messages/MessageElement.cpp b/src/messages/MessageElement.cpp
@@ -208,8 +208,8 @@ void EmoteElement::addToContainer(MessageLayoutContainer &container,
     {
         if (flags.has(MessageElementFlag::EmoteImages))
         {
-            auto image =
-                this->emote_->images.getImageOrLoaded(container.getScale());
+            auto image = this->emote_->images.getImageOrLoaded(
+                container.getImageScale());
             if (image->isEmpty())
                 return;
 
@@ -262,7 +262,7 @@ void LayeredEmoteElement::addToContainer(MessageLayoutContainer &container,
     {
         if (flags.has(MessageElementFlag::EmoteImages))
         {
-            auto images = this->getLoadedImages(container.getScale());
+            auto images = this->getLoadedImages(container.getImageScale());
             if (images.empty())
             {
                 return;
@@ -417,7 +417,7 @@ void BadgeElement::addToContainer(MessageLayoutContainer &container,
     if (flags.hasAny(this->getFlags()))
     {
         auto image =
-            this->emote_->images.getImageOrLoaded(container.getScale());
+            this->emote_->images.getImageOrLoaded(container.getImageScale());
         if (image->isEmpty())
             return;
 
@@ -691,7 +691,7 @@ void SingleLineTextElement::addToContainer(MessageLayoutContainer &container,
             {
                 auto emote = boost::get<EmotePtr>(parsedWord);
                 auto image =
-                    emote->images.getImageOrLoaded(container.getScale());
+                    emote->images.getImageOrLoaded(container.getImageScale());
                 if (!image->isEmpty())
                 {
                     auto emoteScale = getSettings()->emoteScale.getValue();
@@ -828,7 +828,7 @@ void ScalingImageElement::addToContainer(MessageLayoutContainer &container,
     if (flags.hasAny(this->getFlags()))
     {
         const auto &image =
-            this->images_.getImageOrLoaded(container.getScale());
+            this->images_.getImageOrLoaded(container.getImageScale());
         if (image->isEmpty())
             return;
 

diff --git a/src/messages/layouts/MessageLayout.cpp b/src/messages/layouts/MessageLayout.cpp
@@ -74,7 +74,8 @@ int MessageLayout::getWidth() const
 
 // Layout
 // return true if redraw is required
-bool MessageLayout::layout(int width, float scale, MessageElementFlags flags)
+bool MessageLayout::layout(int width, float scale, float imageScale,
+                           MessageElementFlags flags)
 {
     //    BenchmarkGuard benchmark("MessageLayout::layout()");
 
@@ -106,6 +107,8 @@ bool MessageLayout::layout(int width, float scale, MessageElementFlags flags)
     // check if dpi changed
     layoutRequired |= this->scale_ != scale;
     this->scale_ = scale;
+    layoutRequired |= this->imageScale_ != imageScale;
+    this->imageScale_ = imageScale;
 
     if (!layoutRequired)
     {
@@ -143,7 +146,8 @@ void MessageLayout::actuallyLayout(int width, MessageElementFlags flags)
     bool hideSimilar = getSettings()->hideSimilar;
     bool hideReplies = !flags.has(MessageElementFlag::RepliedMessage);
 
-    this->container_.beginLayout(width, this->scale_, messageFlags);
+    this->container_.beginLayout(width, this->scale_, this->imageScale_,
+                                 messageFlags);
 
     for (const auto &element : this->message_->elements)
     {
@@ -280,16 +284,11 @@ QPixmap *MessageLayout::ensureBuffer(QPainter &painter, int width)
     }
 
     // Create new buffer
-#if defined(Q_OS_MACOS) || defined(Q_OS_LINUX)
     this->buffer_ = std::make_unique<QPixmap>(
         int(width * painter.device()->devicePixelRatioF()),
         int(this->container_.getHeight() *
             painter.device()->devicePixelRatioF()));
     this->buffer_->setDevicePixelRatio(painter.device()->devicePixelRatioF());
-#else
-    this->buffer_ = std::make_unique<QPixmap>(
-        width, std::max(16, this->container_.getHeight()));
-#endif
 
     this->bufferValid_ = false;
     DebugCount::increase("message drawing buffers");

diff --git a/src/messages/layouts/MessageLayout.hpp b/src/messages/layouts/MessageLayout.hpp
@@ -52,7 +52,8 @@ class MessageLayout
 
     MessageLayoutFlags flags;
 
-    bool layout(int width, float scale_, MessageElementFlags flags);
+    bool layout(int width, float scale_, float imageScale_,
+                MessageElementFlags flags);
 
     // Painting
     void paint(const MessagePaintContext &ctx);
@@ -105,6 +106,7 @@ class MessageLayout
     int currentLayoutWidth_ = -1;
     int layoutState_ = -1;
     float scale_ = -1;
+    float imageScale_ = -1.F;
     MessageElementFlags currentWordFlags_;
 
 #ifdef FOURTF

diff --git a/src/messages/layouts/MessageLayoutContainer.cpp b/src/messages/layouts/MessageLayoutContainer.cpp
@@ -30,7 +30,7 @@ constexpr const QMargins MARGIN{8, 4, 8, 4};
 namespace chatterino {
 
 void MessageLayoutContainer::beginLayout(int width, float scale,
-                                         MessageFlags flags)
+                                         float imageScale, MessageFlags flags)
 {
     this->elements_.clear();
     this->lines_.clear();
@@ -45,6 +45,7 @@ void MessageLayoutContainer::beginLayout(int width, float scale,
     this->width_ = width;
     this->height_ = 0;
     this->scale_ = scale;
+    this->imageScale_ = imageScale;
     this->flags_ = flags;
     auto mediumFontMetrics =
         getApp()->fonts->getFontMetrics(FontStyle::ChatMedium, scale);
@@ -477,6 +478,11 @@ float MessageLayoutContainer::getScale() const
     return this->scale_;
 }
 
+float MessageLayoutContainer::getImageScale() const
+{
+    return this->imageScale_;
+}
+
 bool MessageLayoutContainer::isCollapsed() const
 {
     return this->isCollapsed_;

diff --git a/src/messages/layouts/MessageLayoutContainer.hpp b/src/messages/layouts/MessageLayoutContainer.hpp
@@ -32,7 +32,8 @@ struct MessageLayoutContainer {
      * This will reset all line calculations, and will be considered incomplete
      * until the accompanying end function has been called
      */
-    void beginLayout(int width_, float scale_, MessageFlags flags_);
+    void beginLayout(int width, float scale, float imageScale,
+                     MessageFlags flags);
 
     /**
      * Finish the layout process of this message
@@ -131,6 +132,11 @@ struct MessageLayoutContainer {
      */
     float getScale() const;
 
+    /**
+     * Returns the image scale
+     */
+    float getImageScale() const;
+
     /**
      * Returns true if this message is collapsed
      */
@@ -250,6 +256,10 @@ struct MessageLayoutContainer {
 
     // variables
     float scale_ = 1.F;
+    /**
+     * Scale factor for images
+     */
+    float imageScale_ = 1.F;
     int width_ = 0;
     MessageFlags flags_{};
     /**

diff --git a/src/widgets/AttachedWindow.cpp b/src/widgets/AttachedWindow.cpp
@@ -271,20 +271,22 @@ void AttachedWindow::updateWindowRect(void *_attachedPtr)
     }
 
     float scale = 1.f;
+    float ourScale = 1.F;
     if (auto dpi = getWindowDpi(attached))
     {
         scale = dpi.get() / 96.f;
+        ourScale = scale / this->devicePixelRatio();
 
         for (auto w : this->ui_.split->findChildren<BaseWidget *>())
         {
-            w->setOverrideScale(scale);
+            w->setOverrideScale(ourScale);
         }
-        this->ui_.split->setOverrideScale(scale);
+        this->ui_.split->setOverrideScale(ourScale);
     }
 
     if (this->height_ != -1)
     {
-        this->ui_.split->setFixedWidth(int(this->width_ * scale));
+        this->ui_.split->setFixedWidth(int(this->width_ * ourScale));
 
         // offset
         int o = this->fullscreen_ ? 0 : 8;