psi-4ward / psitransfer

Simple open source self-hosted file sharing solution.
BSD 2-Clause "Simplified" License
1.46k stars 211 forks source link

Changed the colors of the dark theme to more pleasant ones #227

Closed lzrdblzzrd closed 2 years ago

lzrdblzzrd commented 2 years ago

I couldn't change the color of the svg icons. Because of this, they do not look very nice on a dark background. If you change their color to a lighter one (It is desirable that the icons on the gray colors be #b9bbbe, and on the colors of the purple and green buttons - #ffffff), then everything should look beautiful.

psi-4ward commented 2 years ago

fill is the prop for svgs

but I still don't like the dark theme much and the whole bootstrap stuff could be replaced by some lighter definitions using css-vars

lzrdblzzrd commented 2 years ago

fill is the prop for svgs

I don't really understand how to do this with css.

but I still don't like the dark theme much and the whole bootstrap stuff could be replaced by some lighter definitions using css-vars

I will wait if you suddenly find a replacement for bootstrap, if anything, I can correct the colors if necessary.

psi-4ward commented 2 years ago

https://github.com/psi-4ward/psitransfer/blob/master/public/assets/styles.css#L295

I dont think I'll take the time to rewrite the CSS.

lzrdblzzrd commented 2 years ago

https://github.com/psi-4ward/psitransfer/blob/master/public/assets/styles.css#L295

I dont think I'll take the time to rewrite the CSS.

Ah, got it. I will fix the icon colors soon.

psi-4ward commented 2 years ago

I think the colors are ok for now.

lzrdblzzrd commented 2 years ago

I think the colors are ok for now.

image These colors do not match the color scheme and look somehow out of tune. They should be light in order to be pleasant to look at. That's all that's left to change in colors, and I won't touch styles.css anymore

psi-4ward commented 2 years ago

Pls try the latest master: ac18a15832023db19824368074f78cbbb0b12c3c

lzrdblzzrd commented 2 years ago

Pls try the latest master: ac18a15

Yes, everything looks really good now. Thanks.