Differential D29480

Fix Logarithmic vertical axis has multiple zero labels
ClosedPublic
Actions

Authored by rszczesiak on May 6 2020, 12:05 PM.

Details

Reviewers
tbaumgart
Commits
R261:462f3dcb1306: Fix Logarithmic vertical axis has multiple zero labels
R261:b9d999e137f0: Fix Logarithmic vertical axis has multiple zero labels
Summary

Vertical axis range is now cropped to the value of precision limit.

BUG: 421105

Diff Detail

Repository
R261 KMyMoney
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
rszczesiak requested review of this revision.May 6 2020, 12:05 PM
rszczesiak created this revision.
rszczesiak added a reviewer: tbaumgart.May 6 2020, 12:07 PM
rszczesiak added a project: KMyMoney.
rszczesiak added a comment.May 6 2020, 1:37 PM

Before fix:

After fix:

tbaumgart accepted this revision.May 6 2020, 4:49 PM

I see, you get a kick out of this charting stuff. Cool.

This revision is now accepted and ready to land.May 6 2020, 4:49 PM
rszczesiak added a comment.May 6 2020, 8:01 PM

Thanks :) I really like to see that charts work as expected.

Would you please land this patch for me? I believe the below output from arc means I am not eligible to push to the repo:

 PUSHING  Pushing changes to "origin/master".
fatal: remote error: service not enabled: /kmymoney
Usage Exception: Push failed! Fix the error and run "arc land" again.
Closed by commit R261:462f3dcb1306: Fix Logarithmic vertical axis has multiple zero labels (authored by rszczesiak, committed by tbaumgart). · Explain WhyMay 8 2020, 7:37 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 82242

View Options

kmymoney/plugins/views/reports/core/kreportchartview.h

Loading...
View Options

kmymoney/plugins/views/reports/core/kreportchartview.cpp

Loading...