-
-
Notifications
You must be signed in to change notification settings - Fork 208
Conversation
@huyenltnguyen No need to feel bad, if that is the fix then that is great. Have those changes been pushed up to this branch? |
Don't feel bad, I just wasn't sure if we wanted to continue using the custom theme or vs-dark for the editor, so my original fix was just to remove the offending lines and leave the rest as is. I've now removed what I believe to be the rest of the css that was used to create the custom dark theme for the editor. |
Perfect. Thanks for making these changes. This LGTM |
Thanks @huyenltnguyen and @thatnini 👍 |
Thanks @thatnini . |
Hey @huyenltnguyen @thatnini. I was the one who implemented the colors for the code editor. Don't feel bad. There were a lot of issues with my implementation. I understand why you had to take my styles out. It works better this way. @Bouncey @raisedadead. I am so sorry for all the issues I caused with night mode. I didn't qa my code as properly as I should have. It was one of my first open source contributions, and I was too eager to design the code editor. I'll stick to more of the beginner issues from now on to get a better understanding of the codebase. |
@renojvarghese @thatnini @huyenltnguyen Do not worry, I would rather recommend on working on anything that you wish and like. This is Git. Everything is detectable and reversible. So breaking things is encouraged and OKAY. This is the only way we can learn and ensure good quality. Thanks a lot for for working on the getting the night mode working. Keep up the good work. Cheers and Happy Contributing. |
* fix: night mode tweaks * fix: remove custom dark theme from monaco editor
A couple of tweaks to night mode
Allows the ability to see the selection of text in the editor:
Before
After
Make the test result odd rows darker:
Before
After
Fixes partially freeCodeCamp/freeCodeCamp/#17782