rkflx added a comment.
In D12849#264040 <https://phabricator.kde.org/D12849#264040>, @progwolff wrote: > Not sure if this patch should be landed in this half-finished state. Might be an improvement, but might also be perceived as a regression. > I'm away for some hours now, so if anyone feels that this patch should go in, please feel free to land it. @mart @broulik Should this land now (with possible improvements for the final release), or is this for 5.14 (meaning we'll change around the UI //again//)? --- > In D12849#263865 <https://phabricator.kde.org/D12849#263865>, @rkflx wrote: > >> share/kpackage/kcms/kcm_fonts/contents/ui/main.qml:273:17: QML SpinBox: Binding loop detected for property "value" >> > > > The second on doesn't show up for me... After much head-scratching I found how to reproduce: Remove the `forceFontDPI` key from `~/.config/kcmfonts` (hint hint <https://community.kde.org/Infrastructure/Phabricator#Perform_QA>). REPOSITORY R119 Plasma Desktop BRANCH fonts_kcm_layout (branched from master) REVISION DETAIL https://phabricator.kde.org/D12849 To: progwolff, mart, abetts, ngraham Cc: broulik, zzag, rkflx, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart