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

Added a Search bar and organized command structure #80

Merged
merged 3 commits into from
Jul 30, 2024

Conversation

afonsofrancof
Copy link
Collaborator

@afonsofrancof afonsofrancof commented Jul 26, 2024

Added a Search bar and organized command structure

Type of Change

New feature

Description

Added a search bar to the top of Linutil.
Also, since the bash/neovim setup are technically part of your dotfiles, but were outside the folder, I decided to bring them all together into a node on the tree called "Applications Setup", which makes more sense for a normal user just wanting to set the applications up without having to worry why there are "normal" setups and your dotfiles setups.

Screen.Recording.2024-07-26.at.16.11.27.mov

Testing

I have tested it and everything seems to be working fine.

Impact

If the project grew a lot, it would be hard to find the stuff we want. This makes it easier

Additional Information

@ChrisTitusTech I thought of organizing the applications into categories folders (Terminal, shell, editor, etc) but decided I didn't want to change too much of the current structure and will leave it up to you if you want me to do it.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@ChrisTitusTech
Copy link
Owner

That is very cool! Thanks @afonsofrancof

@afonsofrancof
Copy link
Collaborator Author

afonsofrancof commented Jul 29, 2024

That is very cool! Thanks @afonsofrancof

Thanks 😄.
Btw @ChrisTitusTech , feel free to change the colors of the inactive and active search bar before the merge. The grey and white I used are just placeholders cause I didn't know which ones you wanted.

@JustLinuxUser
Copy link
Contributor

I would also say wait until #82 gets merged / regected, it's a bigger PR, that will change this PR a little, I know yours was first, but that one is much more global then this new feature

@afonsofrancof
Copy link
Collaborator Author

afonsofrancof commented Jul 29, 2024

I would also say wait until #82 gets merged / regected, it's a bigger PR, that will change this PR a little, I know yours was first, but that one is much more global then this new feature

Yup, I think it would be an easy merge.

@ChrisTitusTech ChrisTitusTech merged commit de96983 into ChrisTitusTech:main Jul 30, 2024
2 checks passed
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.

3 participants