Differential D12571 Diff 36299 src/settings/viewmodes/viewsettingstab.cpp

Changeset View

Standalone View

View Options

src/settings/viewmodes/viewsettingstab.cpp

Show All 19 Lines
20#include "viewsettingstab.h" 20​#include "viewsettingstab.h"
21 21​
22#include "dolphin_compactmodesettings.h" 22​#include "dolphin_compactmodesettings.h"
23#include "dolphin_detailsmodesettings.h" 23​#include "dolphin_detailsmodesettings.h"
24#include "dolphin_iconsmodesettings.h" 24​#include "dolphin_iconsmodesettings.h"
25#include "dolphinfontrequester.h" 25​#include "dolphinfontrequester.h"
26#include "views/zoomlevelinfo.h" 26​#include "views/zoomlevelinfo.h"
27 27​
28#include <KComboBox> ​
29#include <KLocalizedString> 28​#include <KLocalizedString>
30 29​
31#include <QApplication> 30​#include <QApplication>
32#include <QCheckBox> 31​#include <QCheckBox>
32​#include <QComboBox>
33#include <QGroupBox> 33​#include <QGroupBox>
34#include <QHelpEvent> 34​#include <QHelpEvent>
35#include <QLabel> 35​#include <QLabel>
36#include <QVBoxLayout> 36​#include <QFormLayout>
37 37​
38ViewSettingsTab::ViewSettingsTab(Mode mode, QWidget* parent) : 38​ViewSettingsTab::ViewSettingsTab(Mode mode, QWidget* parent) :
39 QWidget(parent), 39​ QWidget(parent),
40 m_mode(mode), 40​ m_mode(mode),
41 m_defaultSizeSlider(nullptr), 41​ m_defaultSizeSlider(nullptr),
42 m_previewSizeSlider(nullptr), 42​ m_previewSizeSlider(nullptr),
43 m_fontRequester(nullptr), 43​ m_fontRequester(nullptr),
44 m_widthBox(nullptr), 44​ m_widthBox(nullptr),
45 m_maxLinesBox(nullptr), 45​ m_maxLinesBox(nullptr),
46 m_expandableFolders(nullptr) 46​ m_expandableFolders(nullptr)
47{ 47​{
48 QVBoxLayout* topLayout = new QVBoxLayout(this); 48​ QFormLayout* topLayout = new QFormLayout(this);
49 49​
50 // Create "Icon Size" group ​
51 QGroupBox* iconSizeGroup = new QGroupBox(this); ​
52 iconSizeGroup->setTitle(i18nc("@title:group", "Icon Size")); ​
53 50​
51​ // Create "Icon Size" section
54 const int minRange = ZoomLevelInfo::minimumLevel(); 52​ const int minRange = ZoomLevelInfo::minimumLevel();
55 const int maxRange = ZoomLevelInfo::maximumLevel(); 53​ const int maxRange = ZoomLevelInfo::maximumLevel();
56 54​
57 QLabel* defaultLabel = new QLabel(i18nc("@label:listbox", "Default:"), this); 55​ m_defaultSizeSlider = new QSlider(Qt::Horizontal);
58 m_defaultSizeSlider = new QSlider(Qt::Horizontal, this); ​
59 m_defaultSizeSlider->setPageStep(1); 56​ m_defaultSizeSlider->setPageStep(1);
60 m_defaultSizeSlider->setTickPosition(QSlider::TicksBelow); 57​ m_defaultSizeSlider->setTickPosition(QSlider::TicksBelow);
61 m_defaultSizeSlider->setRange(minRange, maxRange); 58​ m_defaultSizeSlider->setRange(minRange, maxRange);
62 connect(m_defaultSizeSlider, &QSlider::valueChanged, 59​ connect(m_defaultSizeSlider, &QSlider::valueChanged,
63 this, &ViewSettingsTab::slotDefaultSliderMoved); 60​ this, &ViewSettingsTab::slotDefaultSliderMoved);
61​ topLayout->addRow(i18nc("@label:listbox", "Default icon size:"), m_defaultSizeSlider);
64 62​
65 QLabel* previewLabel = new QLabel(i18nc("@label:listbox", "Preview:"), this); 63​ m_previewSizeSlider = new QSlider(Qt::Horizontal);
66 m_previewSizeSlider = new QSlider(Qt::Horizontal, this); ​
67 m_previewSizeSlider->setPageStep(1); 64​ m_previewSizeSlider->setPageStep(1);
68 m_previewSizeSlider->setTickPosition(QSlider::TicksBelow); 65​ m_previewSizeSlider->setTickPosition(QSlider::TicksBelow);
69 m_previewSizeSlider->setRange(minRange, maxRange); 66​ m_previewSizeSlider->setRange(minRange, maxRange);
70 connect(m_previewSizeSlider, &QSlider::valueChanged, 67​ connect(m_previewSizeSlider, &QSlider::valueChanged,
71 this, &ViewSettingsTab::slotPreviewSliderMoved); 68​ this, &ViewSettingsTab::slotPreviewSliderMoved);
69​ topLayout->addRow(i18nc("@label:listbox", "Preview icon size:"), m_previewSizeSlider);
70​
71​
72​ topLayout->addItem(new QSpacerItem(0, 18, QSizePolicy::Fixed, QSizePolicy::Fixed));
73​
74​
75​ // Create "Label" section
76​ m_fontRequester = new DolphinFontRequester(this);
77​ topLayout->addRow(i18nc("@label:listbox", "Label font:"), m_fontRequester);
72 78​
73 QGridLayout* layout = new QGridLayout(iconSizeGroup); ​
74 layout->addWidget(defaultLabel, 0, 0, Qt::AlignRight); ​
75 layout->addWidget(m_defaultSizeSlider, 0, 1); ​
76 layout->addWidget(previewLabel, 1, 0, Qt::AlignRight); ​
77 layout->addWidget(m_previewSizeSlider, 1, 1); ​
78 ​
79 // Create "Text" group ​
80 QGroupBox* textGroup = new QGroupBox(i18nc("@title:group", "Text"), this); ​
81 ​
82 QLabel* fontLabel = new QLabel(i18nc("@label:listbox", "Font:"), textGroup); ​
83 m_fontRequester = new DolphinFontRequester(textGroup); ​
84 ​
85 QGridLayout* textGroupLayout = new QGridLayout(textGroup); ​
86 textGroupLayout->addWidget(fontLabel, 0, 0, Qt::AlignRight); ​
87 textGroupLayout->addWidget(m_fontRequester, 0, 1); ​
88 79​
89 switch (m_mode) { 80​ switch (m_mode) {
90 case IconsMode: { 81​ case IconsMode: {
91 QLabel* widthLabel = new QLabel(i18nc("@label:listbox", "Width:"), textGroup); 82​ m_widthBox = new QComboBox();
elvisangelaccioUnsubmitted
Done

Why the change from KComboBox to QComboBox?

elvisangelaccio: Why the change from `KComboBox` to `QComboBox`?
ngrahamAuthorUnsubmitted
Done

Because KComboBox didn't display correctly within a QFormLayout. QComboBox is a drop-in replacement, so it seemed like an acceptable way to fix the issue.

I also found references in commit messages for several other KDE projects to replacing KComboBox with QComboBox, so it seemed to align with the overall code direction anyway.

ngraham: Because `KComboBox` didn't display correctly within a `QFormLayout`. `QComboBox` is a drop-in…
92 m_widthBox = new KComboBox(textGroup); 83​ m_widthBox->addItem(i18nc("@item:inlistbox Label width", "Small"));
93 m_widthBox->addItem(i18nc("@item:inlistbox Text width", "Small")); 84​ m_widthBox->addItem(i18nc("@item:inlistbox Label width", "Medium"));
94 m_widthBox->addItem(i18nc("@item:inlistbox Text width", "Medium")); 85​ m_widthBox->addItem(i18nc("@item:inlistbox Label width", "Large"));
95 m_widthBox->addItem(i18nc("@item:inlistbox Text width", "Large")); 86​ m_widthBox->addItem(i18nc("@item:inlistbox Label width", "Huge"));
96 m_widthBox->addItem(i18nc("@item:inlistbox Text width", "Huge")); 87​ topLayout->addRow(i18nc("@label:listbox", "Label width:"), m_widthBox);
97 88​
98 QLabel* maxLinesLabel = new QLabel(i18nc("@label:listbox", "Maximum lines:"), textGroup); 89​ m_maxLinesBox = new QComboBox();
99 m_maxLinesBox = new KComboBox(textGroup); ​
100 m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "Unlimited")); 90​ m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "Unlimited"));
101 m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "1")); 91​ m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "1"));
102 m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "2")); 92​ m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "2"));
103 m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "3")); 93​ m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "3"));
104 m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "4")); 94​ m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "4"));
105 m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "5")); 95​ m_maxLinesBox->addItem(i18nc("@item:inlistbox Maximum lines", "5"));
106 96​ topLayout->addRow(i18nc("@label:listbox", "Maximum lines:"), m_maxLinesBox);
107 textGroupLayout->addWidget(widthLabel, 2, 0, Qt::AlignRight); ​
108 textGroupLayout->addWidget(m_widthBox, 2, 1); ​
109 textGroupLayout->addWidget(maxLinesLabel, 3, 0, Qt::AlignRight); ​
110 textGroupLayout->addWidget(m_maxLinesBox, 3, 1); ​
111 break; 97​ break;
112 } 98​ }
113 case CompactMode: { 99​ case CompactMode: {
114 QLabel* maxWidthLabel = new QLabel(i18nc("@label:listbox", "Maximum width:"), textGroup); 100​ m_widthBox = new QComboBox();
115 m_widthBox = new KComboBox(textGroup); ​
116 m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Unlimited")); 101​ m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Unlimited"));
117 m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Small")); 102​ m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Small"));
118 m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Medium")); 103​ m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Medium"));
119 m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Large")); 104​ m_widthBox->addItem(i18nc("@item:inlistbox Maximum width", "Large"));
120 105​ topLayout->addRow(i18nc("@label:listbox", "Maximum width:"), m_widthBox);
121 textGroupLayout->addWidget(maxWidthLabel, 2, 0, Qt::AlignRight); ​
122 textGroupLayout->addWidget(m_widthBox, 2, 1); ​
123 break; 106​ break;
124 } 107​ }
125 case DetailsMode: 108​ case DetailsMode:
126 m_expandableFolders = new QCheckBox(i18nc("@option:check", "Expandable folders"), this); 109​ m_expandableFolders = new QCheckBox(i18nc("@option:check", "Expandable"));
elvisangelaccioUnsubmitted
Not Done

How about "Expandable on click" ?

elvisangelaccio: How about "Expandable on click" ?
ngrahamAuthorUnsubmitted
Not Done

That wording would imply that the folders themselves are clickable to expand them, which isn't accurate.

ngraham: That wording would imply that the folders themselves are clickable to expand them, which isn't…
110​ topLayout->addRow(i18nc("@label:checkbox", "Folders:"), m_expandableFolders);
127 break; 111​ break;
128 default: 112​ default:
129 break; 113​ break;
130 } 114​ }
131 115​
132 topLayout->addWidget(iconSizeGroup); ​
133 topLayout->addWidget(textGroup); ​
134 if (m_expandableFolders) { ​
135 topLayout->addWidget(m_expandableFolders); ​
136 } ​
137 topLayout->addStretch(1); ​
138 ​
139 loadSettings(); 116​ loadSettings();
140 117​
141 connect(m_defaultSizeSlider, &QSlider::valueChanged, this, &ViewSettingsTab::changed); 118​ connect(m_defaultSizeSlider, &QSlider::valueChanged, this, &ViewSettingsTab::changed);
142 connect(m_previewSizeSlider, &QSlider::valueChanged, this, &ViewSettingsTab::changed); 119​ connect(m_previewSizeSlider, &QSlider::valueChanged, this, &ViewSettingsTab::changed);
143 connect(m_fontRequester, &DolphinFontRequester::changed, this, &ViewSettingsTab::changed); 120​ connect(m_fontRequester, &DolphinFontRequester::changed, this, &ViewSettingsTab::changed);
144 121​
145 switch (m_mode) { 122​ switch (m_mode) {
146 case IconsMode: 123​ case IconsMode:
147 connect(m_widthBox, static_cast<void(KComboBox::*)(int)>(&KComboBox::currentIndexChanged), this, &ViewSettingsTab::changed); 124​ connect(m_widthBox, static_cast<void(QComboBox::*)(int)>(&QComboBox::currentIndexChanged), this, &ViewSettingsTab::changed);
148 connect(m_maxLinesBox, static_cast<void(KComboBox::*)(int)>(&KComboBox::currentIndexChanged), this, &ViewSettingsTab::changed); 125​ connect(m_maxLinesBox, static_cast<void(QComboBox::*)(int)>(&QComboBox::currentIndexChanged), this, &ViewSettingsTab::changed);
149 break; 126​ break;
150 case CompactMode: 127​ case CompactMode:
151 connect(m_widthBox, static_cast<void(KComboBox::*)(int)>(&KComboBox::currentIndexChanged), this, &ViewSettingsTab::changed); 128​ connect(m_widthBox, static_cast<void(QComboBox::*)(int)>(&QComboBox::currentIndexChanged), this, &ViewSettingsTab::changed);
152 break; 129​ break;
153 case DetailsMode: 130​ case DetailsMode:
154 connect(m_expandableFolders, &QCheckBox::toggled, this, &ViewSettingsTab::changed); 131​ connect(m_expandableFolders, &QCheckBox::toggled, this, &ViewSettingsTab::changed);
155 break; 132​ break;
156 default: 133​ default:
157 break; 134​ break;
158 } 135​ }
159} 136​}
▲ Show 20 Lines • Show All 132 Lines • Show Last 20 Lines