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

Option to stay scrolled to the bottom when command finishes #244

Open
net opened this issue Oct 17, 2024 · 2 comments
Open

Option to stay scrolled to the bottom when command finishes #244

net opened this issue Oct 17, 2024 · 2 comments

Comments

@net
Copy link

net commented Oct 17, 2024

Auto-scroll doesn't work for me. The output always remains at the top, and I have to press G after running to jump to the bottom.

Correction: auto-scroll works, but Bacon jumps to the top when the command finishes running. An option to stay at the bottom would be great!

Bacon 3.0.0

@net net changed the title Auto-scroll not working Option to stay scrolled to the bottom when command finishes Oct 17, 2024
@Canop
Copy link
Owner

Canop commented Oct 22, 2024

When there's a long output during compilation, autoscroll to the bottom automatically happens. In such case, without user action, you see the output auto scrolling, then the whole is replaced by the report, and you see its top, most interesting part.

Autoscroll detection purpose is to detect when you scrolled to read something which isn't at the bottom, so that it doesn't get away.

The problem is we can't really detect that you want to be at the bottom and stay there after the scrolling output is replaced.
What interaction do you suggest ?

@net
Copy link
Author

net commented Oct 22, 2024

The top is not always the interesting part:

[INFO]: 🎯  Checking for the Wasm target...
[INFO]: 🌀  Compiling to Wasm...
    Finished `release` profile [optimized] target(s) in 0.02s
[INFO]: ⬇️   Installing wasm-bindgen...
[INFO]: Optimizing wasm binaries with `wasm-opt`...
[INFO]: Optional fields missing from Cargo.toml: 'description', 'repository', and 'license'. These are not necessary, but recommended
[INFO]: ✨   Done in 1.08s
[INFO]: 📦   Your wasm pkg is ready to publish at ...
watchman warning:  Recrawled this watch 29 times, most recently because:
MustScanSubDirs UserDroppedTo resolve, please review the information on
https://facebook.github.io/watchman/docs/troubleshooting.html#recrawl
To clear this warning, run:
`watchman watch-del ... ; watchman watch-project ...`

 PASS  js/index.test.ts
  findOccurances
    ✓ should work (3 ms)
    ✓ supports word boundaries (1 ms)
    ✓ supports syllable boundaries (1 ms)

Test Suites: 1 passed, 1 total
Tests:       3 passed, 3 total
Snapshots:   0 total
Time:        0.259 s, estimated 1 s
Ran all test suites.

I suggest a scroll_to_top_on_finish job property that defaults to true, that when false disables the jump to the top when the command finishes and keeps your scrolled-to place instead.

Or, if the way bacon works doesn't really support the idea of keeping your scolled-to place, then a scroll_to_on_finish property that can be top or bottom.

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

No branches or pull requests

2 participants