Version: 1.4.0_svn_647607 (using KDE KDE 3.5.6) Installed from: Compiled From Sources Compiler: gcc (GCC) 4.1.2 OS: Linux The Up and Down arrow buttons in the Content tab of the Edut Plot dialog move selected curve(s) up or down two slots instead of one, as expected. Steps to reproduce: 1. Plot many curves, say: C1, C2, C3, C4, C5 2. Edit Plot -> Content 3. Select one curve, say, C4 and move it, say, up. Results: The resulting order will be: C1, C4, C2, C3, C5 (ie. C4 moved up two slots) Expected results: The resulting order should be: C1, C2, C4, C3, C5 (ie. C4 moves up one slot)
Created attachment 20235 [details] Proposed patch Patch removes duplicate connections.
Interesting. These duplicate connections date back to when the file was created. Looks like they should go.
SVN commit 653138 by arwalker: BUG:144050 Remove duplicate connections so dialog functions correctly M +0 -11 view2dplotwidget.ui.h --- branches/work/kst/1.5/kst/src/libkstapp/view2dplotwidget.ui.h #653137:653138 @@ -30,17 +30,6 @@ _plot = 0L; - connect(DisplayedCurveList, SIGNAL(clicked(QListBoxItem*)), this, SLOT(updateButtons())); - connect(AvailableCurveList, SIGNAL(clicked(QListBoxItem*)), this, SLOT(updateButtons())); - connect(DisplayedCurveList, SIGNAL(doubleClicked(QListBoxItem*)), this, SLOT(removeDisplayedCurve())); - connect(AvailableCurveList, SIGNAL(doubleClicked(QListBoxItem*)), this, SLOT(addDisplayedCurve())); - connect(DisplayedCurveList, SIGNAL(selectionChanged()), this, SLOT(updateButtons())); - connect(AvailableCurveList, SIGNAL(selectionChanged()), this, SLOT(updateButtons())); - connect(_remove, SIGNAL(clicked()), this, SLOT(removeDisplayedCurve())); - connect(_add, SIGNAL(clicked()), this, SLOT(addDisplayedCurve())); - connect(_up, SIGNAL(clicked()), DisplayedCurveList, SLOT(up())); - connect(_down, SIGNAL(clicked()), DisplayedCurveList, SLOT(down())); - _up->setPixmap(BarIcon("up")); _up->setEnabled(false); _up->setAccel(ALT+Key_Up);