codysherman / 2nfm

Share your screen and computer's audio via WebRTC!
https://2n.fm
BSD 3-Clause "New" or "Revised" License
31 stars 16 forks source link

Missing text in dark mode #238

Closed codysherman closed 2 years ago

codysherman commented 2 years ago

Screenshot 2021-12-15 1 52 36 PM

codysherman commented 2 years ago

I highlighted the text to show it.

Carlos-Gaxiola commented 2 years ago

Hello, can I work on this?

codysherman commented 2 years ago

Hi @Carlos-Gaxiola thanks for you interest! If you fork this project and create a branch, you can fix the issue on your branch and then submit a Pull Request on GitHub with your fix.

The set-up instructions are in the README once you've forked and pulled to your machine to get it running locally on localhost.

Hopefully that all makes sense!

A quick note: Due to TURN server private keys not included in this repo, you may find it hard to connect to your own machine's 2N rooms if you're on the the same network. Aka, using one laptop to make a room and another to listen both on the same Wi-Fi. This specific ticket is just a CSS fix so that won't affect you, but if you do want to test streaming, try connecting from your cellular service if you find difficulty connecting to your own rooms.

Carlos-Gaxiola commented 2 years ago

Thank you, I will be working on this

codysherman commented 2 years ago

Awesome to hear! To push you in the right direction, I'm guessing that the solution to this will be that the highlighted text is most likely a hard coded color or hard coded Sass variable. It needs to be switched to the appropriate CSS variable so that its color shifts based on whether it's in light or dark mode. You should be able to look at other text on the page and see that they are set to CSS variables.

I realize that before you get that far, you have a quite a bit of diving in to do, learning how the app even works in the first place. But as you start to figure out how everything is working, the above should help point you to the correct solution.

Light/dark theme support is quite new to 2n.fm, so recent commits will likely also point you in the right direction as we've fixed other cases of text not properly swapping.

Carlos-Gaxiola commented 2 years ago

Nice! Thank you for this guideline

Carlos-Gaxiola commented 2 years ago

Hi, I sent a PR to be reviewed

pociust commented 2 years ago

should be live shortly, thanks so much for your help!