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

S U G G E S T I O N S #6

Open
FrenzyExists opened this issue Oct 25, 2021 · 29 comments
Open

S U G G E S T I O N S #6

FrenzyExists opened this issue Oct 25, 2021 · 29 comments
Labels
help wanted Extra attention is needed

Comments

@FrenzyExists
Copy link
Owner

Heyy Sup! Decided to open an issue here which is more of a Suggestions forum, drop any crazy ideas for the theme here!

@FrenzyExists FrenzyExists added the help wanted Extra attention is needed label Oct 25, 2021
@bakageddy
Copy link
Contributor

Tree sitter support would be awesome! If you plan on porting this scheme to nvim, I can learn and help you create that port. Thanks!

@bakageddy
Copy link
Contributor

bakageddy commented Oct 26, 2021

and if you want to add some features, I would be more than glad to help you out.

@bakageddy
Copy link
Contributor

Feature Idea: Add Visual Inverse Selection, like the one from morhetz/gruvbox.

@FrenzyExists
Copy link
Owner Author

FrenzyExists commented Oct 26, 2021

#6 (comment)

I haven't heard of the visual inverse selection before, i'll give a look to how gruvbox does this 👍

About Treesitter, I think is a good idea, tho, considering how nvim is doing, a Lua version of the aquarium theme would more likely help with this, as Treesitter is in Lua and more and more things in neovim are now being made in lua. I've been working on a nvim/lua version of this theme, tho, still not much. For this repo, maybe a thing or two that are lua focused would be fine, but mostly it should keep itself on vim-made plugins, since hopefully, a lua theme should be around soon

@bakageddy
Copy link
Contributor

Then you can merge my pr when you port this theme to lua, Thank you!

@FrenzyExists
Copy link
Owner Author

No prob 👍

@FrenzyExists
Copy link
Owner Author

Aquarium wasn't really the colors you see now, the original aquarium is now added under the name of Mish, he was the guy who started the idea, and made the first colors. Then I kinda continue working on the theme and adjusted colors as time passed, till what you see today

@FrenzyExists FrenzyExists pinned this issue Oct 31, 2021
@darquill
Copy link

darquill commented Nov 4, 2021

Hey, I made the .Xresources file for the color scheme, maybe you could add it under extras so people can use it say with https://terminal.sexy to export to their terminal of choice config file:

! special
*.foreground:   #c8cedc
*.background:   #20202a
*.cursorColor:  #b8dceb

! black
*.color0:       #3b3b4d
*.color8:       #1b1b23

! red
*.color1:       #ebb9b9
*.color9:       #d95e59

! green
*.color2:       #caf6bb
*.color10:      #8fc587

! yellow
*.color3:       #E6DFB8
*.color11:      #ffcf85

! blue
*.color4:       #cddbf9
*.color12:      #4a83c3

! magenta
*.color5:       #f6bbe7
*.color13:      #bf83b5

! cyan
*.color6:       #b8dceb
*.color14:      #4eb3cd

! white
*.color7:       #c8cedc
*.color15:      #abb2c2

@FrenzyExists
Copy link
Owner Author

Feel free to make a PR 👍

@FrenzyExists
Copy link
Owner Author

@darquill I added the file 👍

@darquill
Copy link

darquill commented Nov 8, 2021

Cheers, I saw the prev message, but then I forgot to make the PR. Glad you added it :) Thanks!

@bakageddy
Copy link
Contributor

Any Idea on porting this theme to emacs?

@FrenzyExists
Copy link
Owner Author

I don't really use emacs, and don't really have any experience with emacs lisp. If someone knows emacs lisp and wants to port it there, it be cool. The PR could be in my dotfiles, which is aquarium themed

@bakageddy
Copy link
Contributor

ok, I will create a pr soon.

@gmcusaro
Copy link

It's not a crazy idea 😅 but, the support for nvim-tree (a file explorer written in Lua) would also be useful.

@FrenzyExists
Copy link
Owner Author

FrenzyExists commented Nov 14, 2021

I actually use nvim-tree, haven't tweaked its colors (yet), I think its actually useful indeed

I could prolly add it, won't be this week i think, college stuff, but afterwards i think i could. If anyone went ahead and implement it one can make a PR

@FrenzyExists
Copy link
Owner Author

FrenzyExists commented Dec 1, 2021

@gmcusaro

News: Aquarium now has NvimTreee support. Give a check in case i forgot something 👍

Welp... not sure what else to do, i guess there's nothing else?

@gmcusaro
Copy link

gmcusaro commented Dec 2, 2021

@FrenzyExists
Great! Everything seems OK with NvimTree!

@gmcusaro
Copy link

gmcusaro commented Dec 5, 2021

During workflow in a room with suffused light and medium/low brightness of the screen I noticed low contrast between the commented code and the background.

The comment color #3D4059, with the background color #20202A have a contrast ration of 1.59:1. One suggestion would be to replace comment color with #44495E with a better ratio of 1.81:1.

@FrenzyExists
Copy link
Owner Author

Before:
image

After:
image

Additional testing with #4b4e67 or 14% brighter from original color:
image

@FrenzyExists
Copy link
Owner Author

FrenzyExists commented Dec 6, 2021

I think i'll go with your color, idk why making the original brighter give this feel that it might blend with the default foreground, yours have a nice contrast that is easy to make what is a default, and a comment

@FrenzyExists
Copy link
Owner Author

FrenzyExists commented Dec 6, 2021

image

Not gonna lie, it makes it much easier to read. Gonna add the changes 👍

Edit: Done!

@gmcusaro
Copy link

gmcusaro commented Dec 6, 2021

@FrenzyExists yeah, too light wouldn't do. It is lighter and a little more gray than blue. Well done!

@jiyometrik
Copy link

support for more neovim-only plugins:

  • barbar
  • bufferline.nvim
  • telescope
  • lspsaga

@FrenzyExists
Copy link
Owner Author

First time I hear of lspsaga. Gonna give a look at em and see what I can do. If anyone wants to get ahead and add support feel free to PR 👍

@oblitum
Copy link

oblitum commented Dec 25, 2022

I don't like undercurls for links... Always used them for errors and spell mistakes, it feels really weird for HTML/Markdown links. Links almost always are underline, so, I wished that wasn't hardcoded.

@nayaabkhan
Copy link

@FrenzyExists Any idea how to make cursorline work? I have enabled it but the current line is not highlighted when using this color scheme.

@rhyses-pieces
Copy link
Contributor

This looks beautiful! Is it okay to port this theme to Wezterm?

@FrenzyExists
Copy link
Owner Author

Sure! Feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

8 participants