View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0016423 | Scribus | User Interface | public | 2021-01-10 09:49 | 2021-03-21 15:16 |
| Reporter | bubu | Assigned To | jghali | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 1.5.7.svn | ||||
| Fixed in Version | 1.5.7.svn | ||||
| Summary | 0016423: Update layout for Gradient Editor | ||||
| Description | In this patch I propose the usual margin and spacing unifying for Gradient Editor. But I need some help to finish it because there is a new area for me: As I know Gradient Editor (gradienteditor) is used in this places: - Edit > Colors and Fills... > Gradients > Add (dialog) - Properties > Colors > Fill/Stroke > Gradient (dock palette) - Properties > Transparency > Fill Color > Gradient (dock palette) The challenge is that it is part of a dialog and part of dock palettes and we have different default margin/spacing for dialogs and for dock palettes. I this case I have to manage only the spacing. I thought that I set spacing in gradienteditor.ui for dock palettes to 3px and in gradientaddedit.cpp I will overwrite it to 6px. But I can't do it, I do knot know how to access "editor" child widget's UI spacing property. Maybe the idea is completely wrong. Please help me, Jean, how to solve this challenge. About this unfinished patch: - I set default margin/spacing values. - The original Edit > Colors and Fills... > Gradients > Add dialog could be over resized. I remove minimumSize property to solve this. | ||||
| Tags | No tags attached. | ||||
| Attached Files | 0001-Update-layout-for-Gradient-Editor.patch (2,370 bytes)
From 6d1aae27f0aa8bc46f3b58aceb6c89cf3b7e9f6f Mon Sep 17 00:00:00 2001
Message-Id: <6d1aae27f0aa8bc46f3b58aceb6c89cf3b7e9f6f.1610146353.git.bubu@ujevangelizacio.hu>
From: =?UTF-8?q?Gyuris=20Gell=C3=A9rt?= <bubu@ujevangelizacio.hu>
Date: Fri, 8 Jan 2021 23:52:21 +0100
Subject: [PATCH] Update layout for Gradient Editor
---
scribus/ui/gradientaddedit.ui | 18 ++++++++++++------
scribus/ui/gradienteditor.ui | 10 +++++-----
2 files changed, 17 insertions(+), 11 deletions(-)
diff --git a/scribus/ui/gradientaddedit.ui b/scribus/ui/gradientaddedit.ui
index 010a343f2..cf017411f 100644
--- a/scribus/ui/gradientaddedit.ui
+++ b/scribus/ui/gradientaddedit.ui
@@ -23,6 +23,18 @@
<bool>true</bool>
</property>
<layout class="QVBoxLayout" name="verticalLayout">
+ <property name="leftMargin">
+ <number>9</number>
+ </property>
+ <property name="topMargin">
+ <number>9</number>
+ </property>
+ <property name="rightMargin">
+ <number>9</number>
+ </property>
+ <property name="bottomMargin">
+ <number>9</number>
+ </property>
<item>
<layout class="QHBoxLayout" name="horizontalLayout">
<property name="sizeConstraint">
@@ -48,12 +60,6 @@
<verstretch>0</verstretch>
</sizepolicy>
</property>
- <property name="minimumSize">
- <size>
- <width>210</width>
- <height>180</height>
- </size>
- </property>
<property name="frameShape">
<enum>QFrame::NoFrame</enum>
</property>
diff --git a/scribus/ui/gradienteditor.ui b/scribus/ui/gradienteditor.ui
index 4b146b172..64fde97f8 100644
--- a/scribus/ui/gradienteditor.ui
+++ b/scribus/ui/gradienteditor.ui
@@ -21,19 +21,19 @@
</property>
<layout class="QGridLayout" name="gridLayout">
<property name="leftMargin">
- <number>4</number>
+ <number>0</number>
</property>
<property name="topMargin">
- <number>4</number>
+ <number>0</number>
</property>
<property name="rightMargin">
- <number>4</number>
+ <number>0</number>
</property>
<property name="bottomMargin">
- <number>4</number>
+ <number>0</number>
</property>
<property name="spacing">
- <number>2</number>
+ <number>3</number>
</property>
<item row="0" column="0" colspan="2">
<widget class="GradientPreview" name="Preview">
--
2.27.0
| ||||
| Patch | No | ||||
| child of | 0013906 | acknowledged | Metabug: UI enhancement proposals (1.5.x.svn) |
|
|
Applied, thanks! |
|
|
Thank for committing. Jean, could you answer my question above please? How to manage that gradienteditor is used in a dialog and in dock palettes ? How to set different spacing? |
|
|
Thanks! Fixed in 1.5.7.svn (24399) |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-01-10 09:49 | bubu | New Issue | |
| 2021-01-10 09:49 | bubu | File Added: 0001-Update-layout-for-Gradient-Editor.patch | |
| 2021-01-10 09:49 | bubu | File Added: edit_gradient.gif | |
| 2021-01-10 18:36 | PeterBenedek | Relationship added | child of 0013906 |
| 2021-01-13 01:31 | jghali | Assigned To | => jghali |
| 2021-01-13 01:31 | jghali | Status | new => resolved |
| 2021-01-13 01:31 | jghali | Resolution | open => fixed |
| 2021-01-13 01:31 | jghali | Fixed in Version | => 1.5.7.svn |
| 2021-01-13 01:31 | jghali | Note Added: 0048681 | |
| 2021-01-13 07:22 | bubu | Note Added: 0048686 | |
| 2021-03-21 15:16 | FirasH | Status | resolved => closed |
| 2021-03-21 15:16 | FirasH | Note Added: 0049008 |