Add range tab to chart reports
ClosedPublic

Authored by wojnilowicz on Feb 17 2017, 8:34 PM.

Details

Reviewers
None
Group Reviewers
KMyMoney
Commits
R261:5116e77bc3cd: Add range tab to reports
Summary

Range tab allows setting X and Y range of chart on one tab.
Additionally the patch contains:

  1. logarithmic axis checkbox
  2. separate H and V grid checkboxes

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.
wojnilowicz retitled this revision from to Add range tab to chart reports.
wojnilowicz updated this object.
wojnilowicz edited the test plan for this revision. (Show Details)
wojnilowicz added a reviewer: KMyMoney.
wojnilowicz set the repository for this revision to R261 KMyMoney.
wojnilowicz added a project: KMyMoney.
wojnilowicz added a subscriber: KMyMoney.

I am just curious, why should an app for private finance should have a logarithmic axis? This does not sound useful to me.

I am just curious, why should an app for private finance should have a logarithmic axis? This does not sound useful to me.

BUG:321987

Well this is not very convincing to me, whatever.

Christian - I think the reason is in the original bug "... changes in e.g. investments with large different values are hardly to recognize on a linear scale (e.g. change of 20 to 10 if you have a graph changing from 150 to 70)." I'd probably like to see some actual examples of cases where it makes sense, but I don't see any reason against doing it.

Christian - I think the reason is in the original bug "... changes in e.g. investments with large different values are hardly to recognize on a linear scale (e.g. change of 20 to 10 if you have a graph changing from 150 to 70)." I'd probably like to see some actual examples of cases where it makes sense, but I don't see any reason against doing it.

Hi Jack, hi Łukasz,

sorry I was not very clear about this. I read the bug but it still does not convince me. Maybe because I do not have such large fluctuations and cannot imaging that a logarithmic scale could help if I had. However, I is not my intention to block or delay this commit. I was just curious.

Greetings

Christian - I think the reason is in the original bug "... changes in e.g. investments with large different values are hardly to recognize on a linear scale (e.g. change of 20 to 10 if you have a graph changing from 150 to 70)." I'd probably like to see some actual examples of cases where it makes sense, but I don't see any reason against doing it.

Hi Jack, hi Łukasz,

sorry I was not very clear about this. I read the bug but it still does not convince me. Maybe because I do not have such large fluctuations and cannot imaging that a logarithmic scale could help if I had. However, I is not my intention to block or delay this commit. I was just curious.

Greetings

Hi Christian,
I don't have such large fluctuations as well, but it wasn't hard to implement logarithmic scaling and there was this user's wish, so I did it.

You don't delay this commit, you can only speed it up :)

Greetings
Łukasz

wojnilowicz abandoned this revision.Mar 4 2017, 8:59 AM

Submited.

This revision was automatically updated to reflect the committed changes.