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

Fixed regression #2284 #2286

Closed
wants to merge 2 commits into from

Conversation

GLakovnik
Copy link
Contributor

Fixes #2284

The -B and -V flags now reset each other in hledger-ui. This gives the expected behavior:
State : Normal -> keypress B -> State: B
State : Normal -> keypress V -> State: V
State : B -> keypress V -> State: V
State : B -> keypress B -> State: Normal
State : V -> keypress B -> State: B
State : V -> Keypress B -> State: Normal

@simonmichael
Copy link
Owner

Thank you very much! I have merged without the merge commit, as 255dc01.

Please claim your $50 regression fixer's bounty: https://hledger.org/REGRESSIONS.html#regression-bounty

@simonmichael simonmichael added the ui The hledger-ui tool. label Nov 18, 2024
@GLakovnik GLakovnik deleted the regression-#2284 branch November 18, 2024 20:54
@GLakovnik
Copy link
Contributor Author

Please consider the bounty a donation to the Hledger Foundation, inc.
I can't get myself to accept money for a fix like this :)

@simonmichael
Copy link
Owner

So noted. Thanks @GLakovnik !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui The hledger-ui tool.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

hledger-ui: Cannot switch from "costs view" to "market values view"
2 participants