https://bugs.kde.org/show_bug.cgi?id=443469

Noah Davis <noaha...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/breeze/-/commit/4ef04223 |ma/breeze/commit/586e746208
                   |a756c5a26eb82d4bcd3ce562252 |0900bc8375a7973ca7624574c0b
                   |8ed0b                       |704
         Resolution|---                         |FIXED
             Status|REOPENED                    |RESOLVED

--- Comment #7 from Noah Davis <noaha...@gmail.com> ---
Git commit 586e7462080900bc8375a7973ca7624574c0b704 by Noah Davis.
Committed on 08/10/2021 at 17:00.
Pushed by ngraham into branch 'master'.

kstyle: Add QFocusFrame to non-view/delegate interactive widget

There are 2 parts that contain the bulk of the code in this patch.

Style::event() is used to apply the QFocusFrame to a widget when it gets
focus with a keyboard input related focus reason. If the focused widget
has a focusProxy, this makes sure the QFocusFrame is applied to the
focusProxy instead.

Style::drawFocusFrame() is mostly what you'd expect. It draws a focus
frame based on the type of widget the QFocusFrame was applied to. I had
to do a workaround for QFocusFrame not fully repainting ouside the
bounds of QSliders and QDials when the handle moves though. What I do is
check if `_focusFrame` is defined and then `_focusFrame->update()`.

M  +252  -3    kstyle/breezestyle.cpp
M  +7    -0    kstyle/breezestyle.h

https://invent.kde.org/plasma/breeze/commit/586e7462080900bc8375a7973ca7624574c0b704

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to