> On Okt. 21, 2015, 9:56 nachm., Christoph Feck wrote: > > Raphael, does the crash still happen with Plasma 5.4.2? There is a comment > > at bug 328807 mentioning a related bug 260869 about modality, which got > > fixed for Plasma 5.4.2.
Hi Christoph, yes, the bug is also present in the latest version. In fact, that every sensor-type has its own handling for the settings dialog (not very nice), we have here different issues. bug 328807 affects only the DancingBars (dialog opens already in a modality manner, the problem is here the handling of the ids after applying the dialog (e.g. out of range error)). bug 260869 affects only the FancyPlotter (was changed to modality by me) - Raphael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125416/#review87231 ----------------------------------------------------------- On Okt. 9, 2015, 3:43 nachm., Raphael Kugelmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125416/ > ----------------------------------------------------------- > > (Updated Okt. 9, 2015, 3:43 nachm.) > > > Review request for Plasma. > > > Bugs: 328807 > https://bugs.kde.org/show_bug.cgi?id=328807 > > > Repository: ksysguard > > > Description > ------- > > BugFix for "Bug 328807 - Removing sensor from graph causes KSysGuard crash" > (affected SensorType: DancingBars) > > https://bugs.kde.org/show_bug.cgi?id=328807 > > Two issues fixed: > - crash during removing a bar Sensor > - wrong Sensors is removed > - labels are not set as expected after removing a bar > > > Diffs > ----- > > gui/SensorDisplayLib/BarGraph.cpp ecd6674ea4f66c3c061feb31c6a68e2e7abed885 > gui/SensorDisplayLib/DancingBars.cpp > 13c8a45f4f4ca9dd16f62cdf6295bcd6643d2a12 > gui/SensorDisplayLib/DancingBarsSettings.h > 27df0666db53621ecddb2e4763b26509103f7381 > gui/SensorDisplayLib/DancingBarsSettings.cpp > 6eb9c21b1670b028570e3826620b98e07eac266d > > Diff: https://git.reviewboard.kde.org/r/125416/diff/ > > > Testing > ------- > > Tested > - no crash anymore > - the remaining sensors are the one as expected > > > Thanks, > > Raphael Kugelmann > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
