https://bugs.kde.org/show_bug.cgi?id=382427
--- Comment #12 from Ralf Habacker <ralf.habac...@freenet.de> --- (In reply to NSLW from comment #11) > (In reply to Ralf Habacker from comment #10) > > (In reply to NSLW from comment #9) > > > (In reply to Ralf Habacker from comment #8) > > > > (In reply to NSLW from comment #7) > > > > > No problem here. Aren't the graphs look correct? > > > > > > > > This is because kmymoney from master branch uses KDChart version 2.6.0 > > > > which > > > > has this bug fixed. 4.8 branch uses KDChart version 2.4.3 which is > > > > broken in > > > > this area. > > > > > > Then I would suggest either fix it > > > > I'm mostly ready with this - see at > > https://github.com/rhabacker/kmymoney/commits/4.8-kdchart-2.6. > > Should I open a review request for this or simply merge into 4.8 ? > > Maybe Thomas would like to review this, but if you just removed old one > version and pasted new one version yes > and it works for you, then I think there is nothing to review. I did not changed any major things. From the git log mentioned above you can see that there is one commit with the update, two commits with fixed bugs I was faced on testing (may be useful for master too) and one commit which adds a simple chart test dialog, which makes it easier to debug chart issues in future. Talking about the bugs I guess it would be better to open separate bug reports for it to be able to have it public. Also I will merge the two remaining commits - the update and the test dialog into one commit to solve this bug. -- You are receiving this mail because: You are the assignee for the bug.
