Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

include zero in line charts for area mode #1803

Merged
merged 2 commits into from
May 16, 2024

Conversation

Grulfen
Copy link
Contributor

@Grulfen Grulfen commented May 9, 2024

This change makes the area mode in line charts always include zero. So that the area is not cropped.
In line mode the old behaviour (show 5% below lowest value) is kept.

This makes it easier to see how big a change in an account is in relation the the account size

Line
bild

Area
bild

Grulfen and others added 2 commits May 9, 2024 09:51
This makes it easier to get an understanding if a change in an account
is big of small compared to the total of the account.

For example:

Small change but looks like very big change
    300 :
        :    xxxxxxxxxxx
        :    x..........
        :    x..........
        :    x..........
        :    x..........
        :xxxxx..........
    250 :_______________

Small change looks like small change
    300 :
        :    xxxxxxxxxxx
        :xxxxx..........
        :...............
        :...............
        :...............
        :...............
      0 :_______________
@yagebu
Copy link
Member

yagebu commented May 16, 2024

Yeah, I think this is also a nice way to address this and have both options :)

@yagebu yagebu merged commit 0c81b2e into beancount:main May 16, 2024
16 checks passed
@Grulfen Grulfen deleted the include-zero-in-area-chart branch May 16, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants