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

feat(gui): add base structure #1935

Merged
merged 19 commits into from Apr 11, 2024
Merged

feat(gui): add base structure #1935

merged 19 commits into from Apr 11, 2024

Conversation

ellie
Copy link
Member

@ellie ellie commented Apr 9, 2024

I have a few motivations here, and absolutely understand that not all current users will use or value this

  1. I'd like to make it easier to explore past history, commands, and stats. TUIs can be very powerful, but there are limits to what we can render
  2. I'd like to make it easier to discover and explore features
  3. I'd like to make Atuin more accessible. Especially as we start to add new features like the daemon, running and installing Atuin can be intimidating to new/inexperienced terminal users
  4. While the search UI works well for its purpose, things like dotfiles/kv/etc will be smoother to use this way

Current WIP:
Screenshot 2024-04-09 at 08 57 17

It does use a webview, but it absolutely is not electron. It uses Tauri, which is a Rust backend and system webview frontend. No chrome, no node. We can reuse the Atuin crates, and also have a much smaller footprint.

Checks

  • I am happy for maintainers to push small adjustments to this PR, to speed up the review cycle
  • I have checked that there are no existing pull requests for the same thing

@arcuru
Copy link
Sponsor Contributor

arcuru commented Apr 9, 2024

I haven't run this yet, but this is a great idea.

@ellie
Copy link
Member Author

ellie commented Apr 11, 2024

just gonna remove the tauri backend from the workspace. doesn't make sense to run all the GUI testing along with the rest, especially when it isn't published

I'll restructure after merging this PR. I'll merge as-is - it's fairly functional, but still needs a bunch of work. I don't want this PR to diverge much further

@ellie ellie marked this pull request as ready for review April 11, 2024 15:58
@ellie ellie changed the title [wip] add gui feat(gui): add base structure Apr 11, 2024
@ellie ellie merged commit 6cd4319 into main Apr 11, 2024
17 checks passed
@ellie ellie deleted the ellie/ui branch April 11, 2024 15:59
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