Differential D27540 Diff 76089 src/settingsstatusindicator.cpp

Changeset View

Standalone View

View Options

src/settingsstatusindicator.cpp

  • This file was added.
1​/*
2​ * This file is part of the KDE libraries
3​ * Copyright (C) 2020 Kevin Ottens <kevin.ottens@enioka.com>
4​ *
5​ * This library is free software; you can redistribute it and/or
6​ * modify it under the terms of the GNU Library General Public
7​ * License as published by the Free Software Foundation; either
8​ * version 2 of the License, or (at your option) any later version.
9​ *
10​ * This library is distributed in the hope that it will be useful,
11​ * but WITHOUT ANY WARRANTY; without even the implied warranty of
12​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
13​ * Library General Public License for more details.
14​ *
15​ * You should have received a copy of the GNU Library General Public License
16​ * along with this library; see the file COPYING.LIB. If not, write to
17​ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
18​ * Boston, MA 02110-1301, USA.
19​ */
20​
21​#include "settingsstatusindicator_p.h"
22​
23​#include <QEvent>
24​#include <QIcon>
25​#include <QPainter>
26​#include <QVariant>
27​
28​namespace {
29​constexpr int INDICATOR_ICON_SIZE = 16;
30​constexpr const char *originalWidthProperty = "originalWidth";
31​}
32​
33​SettingStatusIndicator::SettingStatusIndicator(QWidget *parent)
34​ : QWidget(parent)
35​{
36​ setAttribute(Qt::WA_TransparentForMouseEvents);
37​ show();
38​}
39​
40​SettingStatusIndicator::~SettingStatusIndicator()
41​{
42​ setTrackedWidget(nullptr); // This will neatly reset the width offset
43​}
44​
45​void SettingStatusIndicator::setTrackedWidget(QWidget *widget)
46​{
47​ if (m_trackedWidget == widget) {
48​ return;
49​ }
50​
51​ if (m_trackedWidget) {
52​ resetWidthOffset();
53​ m_trackedWidget->removeEventFilter(this);
54​ }
55​
56​ m_trackedWidget = widget;
57​ if (!m_trackedWidget) {
58​ return;
59​ }
60​
61​ forceWidthOffset();
62​ applyLayout();
63​ m_trackedWidget->installEventFilter(this);
64​}
65​
66​bool SettingStatusIndicator::eventFilter(QObject *watched, QEvent *event)
67​{
68​ Q_UNUSED(watched)
69​ if (event->type() == QEvent::Resize && !m_forcingSize) {
70​ forceWidthOffset();
71​ return true;
72​ }
73​
74​ if (event->type() == QEvent::Resize || event->type() == QEvent::Move) {
75​ applyLayout();
76​ }
davidedmundsonUnsubmitted
Done

This is equivalent to calling showFullScreen(), showMaximized(), or setVisible(true), depending on the platform's default behavior for the window flags.

For X and wayland it's setVisible(true)

but we shouldn't count on it.

davidedmundson: >This is equivalent to calling showFullScreen(), showMaximized(), or setVisible(true)…
ervinAuthorUnsubmitted
Not Done

I don't think it matters for widgets which have a parent and not the Qt::Window window flag, but OK, switching to setVisible() instead of show/hide.

ervin: I don't think it matters for widgets which have a parent and not the Qt::Window window flag…
77​ return false;
78​}
79​
80​void SettingStatusIndicator::paintEvent(QPaintEvent *event)
81​{
82​ Q_UNUSED(event)
83​
84​ const auto icon = isChanged() && !isDefaulted() ? QIcon::fromTheme("document-save-as")
85​ : isChanged() ? QIcon::fromTheme("document-save")
86​ : !isDefaulted() ? QIcon::fromTheme("document-edit")
87​ : QIcon();
88​
89​ if (!icon.isNull()) {
90​ QPainter p(this);
91​ auto pos = QPoint(width() - height(), 0);
92​ p.drawPixmap(pos, icon.pixmap(height(), height()));
93​ }
94​}
95​
96​void SettingStatusIndicator::forceWidthOffset()
97​{
98​ m_forcingSize = true;
99​ auto size = m_trackedWidget->size();
100​ m_trackedWidget->setProperty(originalWidthProperty, size.width());
101​ // We actually respect the minimum size hint to avoid the widget looking odd
102​ // worst case scenario the indicator isn't visible if the widget is ending at the very
103​ // edge of the window and is already too small to be resized, it's a very unlikely case
104​ // though, when reaching the edge it's often greedy in space anyway
105​ size.setWidth(qMax(size.width() - INDICATOR_ICON_SIZE, m_trackedWidget->minimumSizeHint().width()));
106​ m_trackedWidget->resize(size);
107​ m_forcingSize = false;
108​}
109​
110​void SettingStatusIndicator::resetWidthOffset()
111​{
112​ m_forcingSize = true;
113​ auto size = m_trackedWidget->size();
114​ size.setWidth(m_trackedWidget->property(originalWidthProperty).toInt());
115​ m_trackedWidget->setProperty(originalWidthProperty, QVariant());
116​ m_trackedWidget->resize(size);
117​ m_forcingSize = false;
118​}
119​
120​void SettingStatusIndicator::applyLayout()
121​{
122​ resize(INDICATOR_ICON_SIZE, INDICATOR_ICON_SIZE);
123​ move(m_trackedWidget->pos() + QPoint(m_trackedWidget->width() + INDICATOR_ICON_SIZE - width(), 0));
124​}
125​
126​bool SettingStatusIndicator::isChanged() const
127​{
128​ return m_changed;
129​}
130​
131​void SettingStatusIndicator::setChanged(bool changed)
132​{
133​ if (m_changed == changed) {
134​ return;
135​ }
136​
137​ m_changed = changed;
138​ update();
139​}
140​
141​bool SettingStatusIndicator::isDefaulted() const
142​{
143​ return m_defaulted;
144​}
145​
146​void SettingStatusIndicator::setDefaulted(bool defaulted)
147​{
148​ if (m_defaulted == defaulted) {
149​ return;
150​ }
151​
152​ m_defaulted = defaulted;
153​ update();
154​}
davidedmundsonUnsubmitted
Done

Can we be sure the tracked widget always has a parent widget?

If someone doesn't use layouts a widget might not have a parent.

davidedmundson: Can we be sure the tracked widget always has a parent widget? If someone doesn't use layouts a…
ervinAuthorUnsubmitted
Not Done

Well that'd mean the tracked widget is a window... it's pretty much an impossibility IMO.

ervin: Well that'd mean the tracked widget is a window... it's pretty much an impossibility IMO.
davidedmundsonUnsubmitted
Done

unused?

davidedmundson: unused?
ervinAuthorUnsubmitted
Not Done

I'm not sure how you end up to this conclusion, it's written two below if we are at the window edge, and it's used in the move call at the end.

ervin: I'm not sure how you end up to this conclusion, it's written two below if we are at the window…
davidedmundsonUnsubmitted
Done

that's not true for the RTL case where the widget is expected to resize.

It would be w->pos().x() + w->width() - widgetExpectedWidth(w)

davidedmundson: that's not true for the RTL case where the widget is expected to resize. It would be w->pos…
ervinAuthorUnsubmitted
Not Done

Either I misunderstood what you meant or you got the math wrong on that one.

What you're proposing (or what I understood you're proposing) breaks the "RTL + widget at edge" case in my tests. The line I wrote is working perfectly fine for my tests with desktoppath and qtquicksettings both in LTR and RTL modes.

ervin: Either I misunderstood what you meant or you got the math wrong on that one. What you're…