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

If simplified_ui is set to true, it doesn't print messages on client startup/exit. #3437

Open
wants to merge 151 commits into
base: main
Choose a base branch
from

Conversation

Tomcat-42
Copy link

Although this is mostly bikeshedding, I believe it helps with UI consistency.

Additionally, this can make the shell startup smoother by removing the annoying "loading zellij" message before the shell rendering.

  • Before:

asciicast

  • After:

asciicast

client startup/exit.

Although this is mostly bikeshedding, I believe it helps with UI consistency.
@Tomcat-42 Tomcat-42 changed the title If simplified_ui is set to tue, it doesn't print messages on client startup/exit. If simplified_ui is set to true, it doesn't print messages on client startup/exit. Nov 9, 2024
imsnif and others added 28 commits November 9, 2024 14:36
* Remove ForeignFunctionEnv wrapper around PluginEnv

This will enable PluginEnv to be the Store context when migrating to
Wasmtime.

* Pass PluginEnv by value to load_plugin_instance

This will allow removing the Clone impl from PluginEnv when migrating to
Wasmtime as required by the missing Clone impl on Wasmtime's WasiCtx.

* Avoid passing a Store around when an Engine is enough

* Pass PluginEnv to the wasi read/write functions

Wasmtime requires storing the read/write end of the pipe outside of the
WasiCtx. Passing PluginEnv to these functions allows storing them in the
PluginEnv.

* Migrate to Wasmtime

* Switch from wasi-common to wasmtime-wasi

* Reduce verbosity of wasmtime_wasi logs

* Increase startup delay

To wait for all plugins to be compiled.

* Disable some wasmtime features

* Update to Wasmtime 21.0.1
* work

* work

* working

* get default mode from server and some ui responsiveness

* work

* finish design and get tests to pass

* get e2e tests to pass

* add classic layout

* add classic layout assets

* fix e2e tests

* style(fmt): rustfmt

* fix plugin system test

* style(fmt): some cleanups
* cargo: Remove `rust-version` (msrv) field

from `Cargo.toml`. This has gone out of sync with the rust version
specified in `rust-toolchain.toml` and appears to have little
consequence to development, since it defines only the MSRV. We're using
`rust-toolchain.toml` to strictly enforce a specific rust version for
development, hence we stick with that.

Closes zellij-org#3476

* CHANGELOG: Add MR zellij-org#3480.
zellij-org#3483)

* fix(terminal): properly cache cursor position for synchronized renders

* style(fmt): rustfmt
* cargo: Reintroduce `rust-version` field

in `Cargo.toml` since it appears that client-side `cargo` will check
this field and [error when the users rust toolchain is too old][1]. Add
comments in `Cargo.toml` and `rust-toolchain.toml` pointing to each
other so the rust versions mentioned in each file are hopefully kept in
sync in the future.

[1]: https://doc.rust-lang.org/cargo/reference/manifest.html#the-rust-version-field

* CHANGELOG: Add MR zellij-org#3482.
…ime (zellij-org#3502)

* rebind => reconfigure

* persist keybinds and mode info to new tabs

* add configuration plugin

* make tests pass

* remove warnings

* style(fmt): rustfmt
* fix(serialization): do not serialize junk sessions

* style(fmt): rustfmt
…org#3505)

* fix(startup): do not parse resurrectable sessions on startup

* style(fmt): rustfmt
rbardini and others added 29 commits November 9, 2024 14:36
Onedark is a popular theme from Atom text editor. 

I've used these sources as the reference for implementing the theme in zellij.
https://github.com/joshdick/onedark.vim
https://www.figma.com/community/file/1137445418485757476/atom-one-dark-color-palette
the previous themes do not have the correct colors and aren't great
for the eyes. when selecting e.g. to copy text, it uses
red for light theme and the same bg for dark theme.

the previous light theme is not even a light theme so I also
fixed that as well.

Signed-off-by: Soc Virnyl Estela <[email protected]>
* fix(startup): try create config folder if it doesn't exist

* fix(configuration): tab bar ui

* fix(configuration): rebind ctrl-s to ctrl-a

* fix(configuration): remove extra rebinding leaders screen
…n hard-coded normal mode (zellij-org#3715)

* fix(keybindings): only forward clear keys in locked mode and default mode

* style(fmt): rustfmt
* fix(input): refix ctrl-j

* fix e2e tests
* fix(plugins): maintain cwd between plugin reloads

* fix(plugin-manager): default to loading plugins in the foreground and allow sending space in configuration
…ellij-org#3750)

* fix(output-buffer): truncate grid height when not rendering it fully

* also fix for cases where the changed lines are not contiguous
@Tomcat-42
Copy link
Author

Hello @imsnif, could you please take a look? 🥺

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.