diff options
| author | Albert Astals Cid <albert.astals.cid@kdab.com> | 2017-12-21 16:55:42 +0100 |
|---|---|---|
| committer | Albert Astals Cid <albert.astals.cid@kdab.com> | 2018-01-05 15:38:35 +0000 |
| commit | 5e2f583a67c75c1c03e213467bb56207e5084279 (patch) | |
| tree | 93e11c9be1849690eea39dafb1bd8373b9f7dbf1 /src/printsupport/dialogs/qprintpropertieswidget.ui | |
| parent | 5e9e4ccdc363297f70d4ebfbbb3e279670eca553 (diff) | |
CUPS: Fix conflict handling
The previous code assumed that ppdMarkOption returning non zero (i.e. it has conflicts)
also meant that the option wasn't applied at the ppd level, but it actually is.
What we need to do is after calling ppdMarkOption parse the tree again looking
to see if any option is conflicting and mark it as such in the UI.
Change-Id: I836f1902d14dc8c176bb06776471cbf4ed11786f
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
Diffstat (limited to 'src/printsupport/dialogs/qprintpropertieswidget.ui')
| -rw-r--r-- | src/printsupport/dialogs/qprintpropertieswidget.ui | 65 |
1 files changed, 59 insertions, 6 deletions
diff --git a/src/printsupport/dialogs/qprintpropertieswidget.ui b/src/printsupport/dialogs/qprintpropertieswidget.ui index 0e56fb33df..d8e526139b 100644 --- a/src/printsupport/dialogs/qprintpropertieswidget.ui +++ b/src/printsupport/dialogs/qprintpropertieswidget.ui @@ -14,7 +14,16 @@ <string>Form</string> </property> <layout class="QVBoxLayout" name="verticalLayout_4"> - <property name="margin"> + <property name="leftMargin"> + <number>0</number> + </property> + <property name="topMargin"> + <number>0</number> + </property> + <property name="rightMargin"> + <number>0</number> + </property> + <property name="bottomMargin"> <number>0</number> </property> <item> @@ -32,18 +41,62 @@ </item> </layout> </widget> - <widget class="QWidget" name="cupsPropertiesPage" > - <attribute name="title" > + <widget class="QWidget" name="cupsPropertiesPage"> + <attribute name="title"> <string>Advanced</string> </attribute> - <layout class="QHBoxLayout" name="horizontalLayout_2" > + <layout class="QVBoxLayout" name="verticalLayout_2"> <item> - <widget class="QTreeView" name="treeView" > - <property name="alternatingRowColors" > + <widget class="QTreeView" name="treeView"> + <property name="alternatingRowColors"> <bool>true</bool> </property> </widget> </item> + <item> + <widget class="QLabel" name="conflictsLabel"> + <property name="palette"> + <palette> + <active> + <colorrole role="WindowText"> + <brush brushstyle="SolidPattern"> + <color alpha="255"> + <red>255</red> + <green>0</green> + <blue>0</blue> + </color> + </brush> + </colorrole> + </active> + <inactive> + <colorrole role="WindowText"> + <brush brushstyle="SolidPattern"> + <color alpha="255"> + <red>255</red> + <green>0</green> + <blue>0</blue> + </color> + </brush> + </colorrole> + </inactive> + <disabled> + <colorrole role="WindowText"> + <brush brushstyle="SolidPattern"> + <color alpha="255"> + <red>165</red> + <green>167</green> + <blue>169</blue> + </color> + </brush> + </colorrole> + </disabled> + </palette> + </property> + <property name="text"> + <string>There are conflicts in some options. Please fix them.</string> + </property> + </widget> + </item> </layout> </widget> </widget> |