Page MenuHomePhabricator

[electrum] scripted-diff: use some Qt6 enums
ClosedPublic

Authored by PiRK on Jul 8 2025, 13:45.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC3430fbdf2349: [electrum] scripted-diff: use some Qt6 enums
Summary
  • QSizePolicy.(\w+) -> QSizePolicy.Policy.$1
  • Qt\.(AlignCenter|AlignHCenter|AlignVCenter|AlignLeft|AlignRight|AlignTop|AlignBottom|AlignJustify|AlignBaseline) -> Qt.AlignmentFlag.$1
  • QLineEdit.Password -> QLineEdit.EchoMode.Password
  • Qt\.(\w+)Focus([^\w]) -> Qt\.FocusPolicy.$1Focus$2

There are a lot more changes required to switch to the Qt6 API, but we don't need to do it all in a single commit thanks to qtpy providing the Qt6 compatibility layer for Qt5
These instances are the ones related to D18338

Depends on D18338

Test Plan
QT_API=pyqt6 ./electrum-abc
QT_API=pyqt5 ./electrum-abc

Diff Detail

Repository
rABC Bitcoin ABC
Branch
trezor_qt6
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33798
Build 67074: Build Diffelectrum-tests · electrum-functional-tests
Build 67073: arc lint + arc unit

Event Timeline

PiRK requested review of this revision.Jul 8 2025, 13:45
PiRK edited the summary of this revision. (Show Details)

don't forget AlignRight

Fabien added a subscriber: Fabien.
Fabien added inline comments.
electrum/electrumabc_plugins/fusion/qt.py
1965

shouldn't this be removed instead ?

This revision is now accepted and ready to land.Jul 8 2025, 13:59

remove commented dead code

This revision was automatically updated to reflect the committed changes.