Differential D7087 Diff 17613 Modules/about-distro/src/Module.ui

Changeset View

Standalone View

View Options

Modules/about-distro/src/Module.ui

Show First 20 Lines • Show All 144 Lines • ▼ Show 20 Line(s)143 <item row="13" column="1">
145 <property name="text"> 145​ <property name="text">
146 <string>Memory:</string> 146​ <string>Memory:</string>
147 </property> 147​ </property>
148 <property name="alignment"> 148​ <property name="alignment">
149 <set>Qt::AlignRight|Qt::AlignTrailing|Qt::AlignVCenter</set> 149​ <set>Qt::AlignRight|Qt::AlignTrailing|Qt::AlignVCenter</set>
150 </property> 150​ </property>
151 </widget> 151​ </widget>
152 </item> 152​ </item>
153 <item row="14" column="3"> 153​ <item row="16" column="3">
154 <spacer name="verticalSpacer"> 154​ <spacer name="verticalSpacer">
155 <property name="orientation"> 155​ <property name="orientation">
156 <enum>Qt::Vertical</enum> 156​ <enum>Qt::Vertical</enum>
157 </property> 157​ </property>
158 <property name="sizeHint" stdset="0"> 158​ <property name="sizeHint" stdset="0">
159 <size> 159​ <size>
160 <width>20</width> 160​ <width>20</width>
161 <height>40</height> 161​ <height>40</height>
▲ Show 20 Lines • Show All 75 Lines • ▼ Show 20 Line(s)235 <item row="7" column="1">
237 <property name="text"> 237​ <property name="text">
238 <string>Qt Version:</string> 238​ <string>Qt Version:</string>
239 </property> 239​ </property>
240 <property name="alignment"> 240​ <property name="alignment">
241 <set>Qt::AlignRight|Qt::AlignTrailing|Qt::AlignVCenter</set> 241​ <set>Qt::AlignRight|Qt::AlignTrailing|Qt::AlignVCenter</set>
242 </property> 242​ </property>
243 </widget> 243​ </widget>
244 </item> 244​ </item>
245​ <item row="15" column="3">
246​ <widget class="QPushButton" name="pushButtonActionMenu">
247​ <property name="sizePolicy">
248​ <sizepolicy hsizetype="Fixed" vsizetype="Fixed">
249​ <horstretch>0</horstretch>
250​ <verstretch>0</verstretch>
251​ </sizepolicy>
252​ </property>
253​ <property name="text">
254​ <string>Menu</string>
255​ </property>
256​ <property name="icon">
257​ <iconset theme="application-menu">
258​ <normaloff>.</normaloff>.</iconset>
259​ </property>
260​ </widget>
261​ </item>
245 <item row="9" column="3"> 262​ <item row="9" column="3">
246 <widget class="QLabel" name="bitsLabel"> 263​ <widget class="QLabel" name="bitsLabel">
247 <property name="text"> 264​ <property name="text">
248 <string notr="true">{QtArchitecutre}</string> 265​ <string notr="true">{QtArchitecutre}</string>
249 </property> 266​ </property>
250 </widget> 267​ </widget>
251 </item> 268​ </item>
252 <item row="11" column="3"> 269​ <item row="11" column="3">
▲ Show 20 Lines • Show All 83 Lines • ▼ Show 20 Line(s)
336 </item> 353​ </item>
337 <item row="6" column="3"> 354​ <item row="6" column="3">
338 <widget class="QLabel" name="frameworksLabel"> 355​ <widget class="QLabel" name="frameworksLabel">
339 <property name="text"> 356​ <property name="text">
340 <string notr="true">{FrameworksVersion}</string> 357​ <string notr="true">{FrameworksVersion}</string>
341 </property> 358​ </property>
342 </widget> 359​ </widget>
343 </item> 360​ </item>
361​ <item row="14" column="3">
362​ <spacer name="verticalSpacer_7">
363​ <property name="orientation">
364​ <enum>Qt::Vertical</enum>
365​ </property>
366​ <property name="sizeType">
367​ <enum>QSizePolicy::Fixed</enum>
368​ </property>
rkflxUnsubmitted
Done

Do you need this? Pressing the Reset button for that property in Qt Designer removes this for me.

rkflx: Do you need this? Pressing the {nav Reset} button for that property in Qt Designer removes this…
gregormiAuthorUnsubmitted
Done

If I remove that part, the button gets wider; I would prefer to keep it as small as possible.

gregormi: If I remove that part, the button gets wider; I would prefer to keep it as small as possible.
rkflxUnsubmitted
Not Done

See reply in main comment.

rkflx: See reply in main comment.
369​ <property name="sizeHint" stdset="0">
370​ <size>
371​ <width>20</width>
372​ <height>24</height>
373​ </size>
374​ </property>
375​ </spacer>
376​ </item>
344 </layout> 377​ </layout>
rkflxUnsubmitted
Done

This looks odd.

rkflx: This looks odd.
345 </widget> 378​ </widget>
346 <resources/> 379​ <resources/>
347 <connections/> 380​ <connections/>
348</ui> 381​</ui>
rkflxUnsubmitted
Done

For me the shortcut actually works, but ideally this should use the standard shortcut for copying, which the user might have set to something different than Ctrl+C.

You could try to look at how it works in Spectacle. I suspect you'd need to add the appropriate action or standard shortcut for that, e.g. KStandardAction::Copy or QKeySequence::Copy, instead of setting shortcut and icon manually.

rkflx: For me the shortcut actually works, but ideally this should use the standard shortcut for…
gregormiAuthorUnsubmitted
Done

The KStandardAction method seems only easily applicable with QToolButtons but not with QPushButton. Furthermore, I got the "ambiguous key binding" when I tried it. I now settled for QKeySequence::Copy which does not work for me but at least results in no error message.

gregormi: The KStandardAction method seems only easily applicable with QToolButtons but not with…
rkflxUnsubmitted
Done

Works perfectly fine for me. I think there might be something off with your local shortcuts setup.

rkflx: Works perfectly fine for me. I think there might be something off with your local shortcuts…
gregormiAuthorUnsubmitted
Not Done

Ok. Then I leave it like that.

gregormi: Ok. Then I leave it like that.