Skip to content
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

Update of FAQ related to characters size #140

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ser
Copy link
Contributor

@ser ser commented Aug 10, 2022

That receipt should work on most of GNU/Linux distributions.

That receipt should work on most of GNU/Linux distributions.
@SomberNight
Copy link
Member

Thanks. Documenting the issue is one approach, but ideally we should try to fix the software so that it works out of the box.

Could you open an issue in the main repo with some screenshots of how the application looks for you, and also, could you try with this patch?

diff --git a/electrum/gui/qt/__init__.py b/electrum/gui/qt/__init__.py
index 3290833019..a922da9074 100644
--- a/electrum/gui/qt/__init__.py
+++ b/electrum/gui/qt/__init__.py
@@ -124,6 +124,8 @@ class ElectrumGui(BaseElectrumGui, Logger):
             QtCore.QCoreApplication.setAttribute(QtCore.Qt.AA_ShareOpenGLContexts)
         if hasattr(QGuiApplication, 'setDesktopFileName'):
             QGuiApplication.setDesktopFileName('electrum.desktop')
+        if hasattr(QtCore.Qt, "AA_EnableHighDpiScaling"):
+            QtCore.QCoreApplication.setAttribute(Qt.AA_EnableHighDpiScaling)
         self.gui_thread = threading.current_thread()
         self.windows = []  # type: List[ElectrumWindow]
         self.efilter = OpenFileEventFilter(self.windows)

(I guess it is similar to setting the QT_AUTO_SCREEN_SCALE_FACTOR=1 env var?)

@ser
Copy link
Contributor Author

ser commented Aug 10, 2022

The patch works correctly setting QT_AUTO_SCREEN_SCALE_FACTOR.

But how to change font size if it's not ideal? I mean QT_FONT_DPI - maybe we can leave that part in the FAQ - as current patch will satisfy 90% of users.

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.

2 participants