|
|
JLuc, are you talking about the 'Nodes' palette menu? |
|
|
JLuc
2014-07-25 08:31
developer
~0032987
Last edited: 2014-07-25 08:37
|
I think so.
It keeps looking pressed + having a semi transparent pink layer on top of it.
When the focus is given to some other window (not the "nodes" dialog anymore), the pink layer disappear but the 1st button keeps looking pressed.
I upload screenshot.
|
|
|
|
That's mostly 'cosmetic'. |
|
|
|
FYI, spelling and grammar issue in
"Open a Polygon or Cuts a bezier curve" |
|
|
|
"Move Nodes" needs to be enabled to use the 3 buttons next to it.
"Move Control Points" need to be enabled for the same reason.
It's not a bug in my opinion, maybe the button does not look very nice when selected, but seems something with Ubuntu UI style.
Never seen that on XFCE (screenshot attached) or KDE. |
|
|
|
The problem is when you click "Open a Polygon or Cuts a Bezier Curve" and click again on "Move Nodes". "Open a Polygon or Cuts a Bezier Curve" in that case should be deselected. |
|
|
|
Added a .patch to fix the problem that kept "Open a Polygon or Cuts a Bezier Curve" selected even if not active. Tested here and works correctly. |
|
|
|
Updated the report. |
|
|
FirasH
2014-08-31 18:28
reporter
|
11400.diff (2,887 bytes)
Index: scribus/ui/nodeeditpalette.cpp
===================================================================
--- scribus/ui/nodeeditpalette.cpp (revisione 19425)
+++ scribus/ui/nodeeditpalette.cpp (copia locale)
@@ -100,8 +100,8 @@
gridLayout->addWidget(Res1Node, 1, 3, 1, 1);
PolySplit = new QToolButton(this);
- PolySplit->setEnabled( false );
- PolySplit->setCheckable( true );
+ PolySplit->setEnabled(false);
+ PolySplit->setCheckable(true);
PolySplit->setIcon(QIcon(loadIcon("PolyCut.png")));
PolySplit->setIconSize(iconSize);
PolySplit->setContentsMargins(0, 0, 0, 0);
@@ -109,7 +109,7 @@
gridLayout->addWidget(PolySplit, 2, 0, 1, 1);
BezierClose = new QToolButton(this);
- BezierClose->setEnabled( false );
+ BezierClose->setEnabled(false);
BezierClose->setIcon(QIcon(loadIcon("BezierClose.png")));
BezierClose->setIconSize(iconSize);
BezierClose->setContentsMargins(0, 0, 0, 0);
@@ -617,6 +617,7 @@
m_doc->nodeEdit.MoveSym = true;
SymMove->setChecked(true);
AsymMove->setChecked(false);
+ PolySplit->setChecked(false);
}
void NodePalette::SetAsym()
@@ -625,6 +626,7 @@
m_doc->nodeEdit.MoveSym = false;
SymMove->setChecked(false);
AsymMove->setChecked(true);
+ PolySplit->setChecked(false);
}
void NodePalette::SetXY(double x, double y)
@@ -819,6 +821,7 @@
MoveControl->setChecked(true);
DeleteNode->setChecked(false);
AddNode->setChecked(false);
+ PolySplit->setChecked(false);
}
void NodePalette::MoveN()
@@ -843,6 +846,7 @@
MoveControl->setChecked(false);
DeleteNode->setChecked(false);
AddNode->setChecked(false);
+ PolySplit->setChecked(false);
}
void NodePalette::AddN()
@@ -857,6 +861,7 @@
Res1Node->setEnabled(false);
AddNode->setChecked(true);
DeleteNode->setChecked(false);
+ PolySplit->setChecked(false);
}
void NodePalette::DelN()
@@ -871,6 +876,7 @@
Res1Node->setEnabled(false);
AddNode->setChecked(false);
DeleteNode->setChecked(true);
+ PolySplit->setChecked(false);
}
void NodePalette::closeEvent(QCloseEvent *ce)
@@ -887,8 +893,8 @@
if (currItem->itemType() == PageItem::PathText)
currItem->updatePolyClip();
}
- PolySplit->setEnabled( false );
- BezierClose->setEnabled( false );
+ PolySplit->setEnabled(false);
+ BezierClose->setEnabled(false);
emit paletteClosed();
ScrPaletteBase::closeEvent(ce);
}
@@ -909,8 +915,8 @@
if (currItem && currItem->itemType() == PageItem::PathText)
currItem->updatePolyClip();
}
- PolySplit->setEnabled( false );
- BezierClose->setEnabled( false );
+ PolySplit->setEnabled(false);
+ BezierClose->setEnabled(false);
EditCont->setChecked(false);
emit paletteClosed();
}
@@ -935,8 +941,8 @@
if (currItem->itemType() == PageItem::PathText)
currItem->updatePolyClip();
}
- PolySplit->setEnabled( false );
- BezierClose->setEnabled( false );
+ PolySplit->setEnabled(false);
+ BezierClose->setEnabled(false);
EditCont->setChecked(false);
emit paletteClosed();
}
|
|
|
|
I've used it for a few days and works correctly.
It would be very nice if after applying the patch to 1.5.0.svn could also be backported to 1.4.5.svn, it's a very simple patch. |
|
|
FirasH
2014-09-10 16:52
reporter
|
|
|
|
|
bumping patch to top of Mantis list |
|
|
|
Thanks cbradney! |
|
