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

Pop!_OS Style #26

Open
adityamwagh opened this issue Aug 9, 2020 · 9 comments
Open

Pop!_OS Style #26

adityamwagh opened this issue Aug 9, 2020 · 9 comments

Comments

@adityamwagh
Copy link
Contributor

adityamwagh commented Aug 9, 2020

I have created a Pop!_OS style for regolith.

I would like to add it here.

@adityamwagh adityamwagh changed the title Pop!_OS style Pop!_OS Style Aug 9, 2020
@adityamwagh
Copy link
Contributor Author

I have forked it at https://github.com/ad1tyawagh/regolith-styles. and changed the colors in color and rofi.rasi to match the Pop!_OS GTK theme.

Since I am using Pop!_OS, I have access to the Pop terminal theme. So we can get it from there.

If anything else needs to be done, let me know.

@kgilmer
Copy link
Member

kgilmer commented Aug 10, 2020

Awesome, that's a great idea @ad1tyawagh ! There are a number of people that use Regolith on Pop!_OS, so I'm sure this will be a welcome addition. I'm a bit backed up on packaging work, let me get back to you in a bit! 😄

@kgilmer
Copy link
Member

kgilmer commented Aug 10, 2020

@ad1tyawagh you can create a PR with your changes so we can review them. Here is an example I just merged: #25

@adityamwagh
Copy link
Contributor Author

Okay! But it will take some time I think.

I am having some trouble porting the theme. I need to change colors to match the desktop perfectly.

About color palette

Also, why did you guys use base03, base02, ..., green as the color palette? Is there a very important reason behind that selection?

It's kind of difficult to port themes that way.

I think we should use color0, color1, ..., color15 as that makes it easy to port themes.

@adityamwagh
Copy link
Contributor Author

I got it to show correct colors finally.

Also, there seems to be an issue with GNOME terminal color schemes in all styles. The cursor foreground color defaults to red.

@kgilmer
Copy link
Member

kgilmer commented Aug 11, 2020

The original key names for the colors came from the Solarized color palette definition, which was the original look of Regolith (before this was configurable) Yes you are right, it would be better to clean that up. It requires some big refactoring to all the looks, and is tedious. No one has picked it up as a task at this point.

Regarding the red cursor in gnome terminal....not sure what you mean. Can you provide a screenshot?

@adityamwagh
Copy link
Contributor Author

I would love to fix key names.

How do I proceed with that?

From what I know, I would have to change the colors and other values in the files:
gnome, i3-wm, i3xrocks, rofi and st-term.

Let me know if there are any other changes to be made, I will make a pull request after knowing exactly what all values are to be changed.

We can push this in the next release, or if I finish it sooner, we can change all the styles to match the new key names.

@kgilmer
Copy link
Member

kgilmer commented Jan 24, 2021

Hi @ad1tyawagh , sorry for the long delay. I'm starting to look into big changes to looks to simlify them and make it easier to create new ones. My thoughts are here if you're still interested.

I am actively experimenting with rethinking how Xresources are used to express UI configuration in Regolith. I can update you when I think I have something worth reviewing, LMK.

@adityamwagh
Copy link
Contributor Author

Sure. Using the base16 color palette is a good idea. And the package organization changes are much appreciated.

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

No branches or pull requests

2 participants