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

Edit scrollback #1456

Merged
merged 22 commits into from Jun 6, 2022
Merged

Edit scrollback #1456

merged 22 commits into from Jun 6, 2022

Conversation

cosminadrianpopescu
Copy link
Contributor

PR for #1403

@imsnif
Copy link
Member

imsnif commented Jun 5, 2022

@a-kenji - any clues as to why the nix build is failing?

@a-kenji
Copy link
Contributor

a-kenji commented Jun 5, 2022

@imsnif,
I assume this is because the Cargo.lock file is in a weird state.
cargo update and committing the lock file might alleviate this issue,
If not I think I can look more into this tomorrow.

@imsnif
Copy link
Member

imsnif commented Jun 5, 2022

@a-kenji - FYI, it works now. The Cargo.lock was updated from a local build (I update and push it all the time as I'm building, testing manually and running tests). What fixed this was merging from main and then building (which indeed included a change to the Cargo.lock file over the one from this branch). Could it be that the nix test uses some sort of external state combined with main?

@imsnif
Copy link
Member

imsnif commented Jun 6, 2022

Thank you for your excellent work on this @cosminadrianpopescu - I'm stoked about this feature and think many others will be as well! Hope to see more contributions from you - even if Rust might not be your cup of tea :)

@imsnif imsnif merged commit e1fcf3a into zellij-org:main Jun 6, 2022
@cosminadrianpopescu
Copy link
Contributor Author

Thank you for your excellent work on this

You welcome.

I'm stoked about this feature and think many others will be as well!

I also think it's a killer feature. This would be enough to make me switch from tmux. And I never had any big issues with tmux's find and copy paste. Just small annoyances. But to be able to open all that buffer in vim, it cannot be beaten by any paradigm impementation. vim is the best at this kind of things.

Hope to see more contributions from you

I was thinking now of finishing this by adding a nice name to the editor panel, like we discussed. Then I was thinking at looking into the buffer's rename (what we discussed before this: adding an option to keep the tab's shortcuts and by default re-naming the tabs when closing one of the tabs which is not the last one). What do you think?

even if Rust might not be your cup of tea

Yeah... I can see the advantages, and I can understand if someone wants to program something for routers for example (with limited resources). But for doing something in the user space (no matter how low level), I think a garbage collector is really nothing compared with a steep learning curve. I think that this steep learning curve is going to make a lot of people just not want to contribute.

@imsnif
Copy link
Member

imsnif commented Jun 6, 2022

I was thinking now of finishing this by adding a nice name to the editor panel, like we discussed. Then I was thinking at looking into the buffer's rename (what we discussed before this: adding an option to keep the tab's shortcuts and by default re-naming the tabs when closing one of the tabs which is not the last one). What do you think?

Sounds great.

@a-kenji
Copy link
Contributor

a-kenji commented Jun 6, 2022

@cosminadrianpopescu
I agree, this is awesome, thanks for all the work.

@imsnif,
The nix build shouldn't use external state from main, but it might be possible, I will look into it a bit more.
Basically it uses the nix files, the Cargo.toml, Cargo.lock and the rust-toolchain.toml, which should be contained inside the branch.

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

3 participants