Skip to content
This repository has been archived by the owner on Oct 16, 2019. It is now read-only.

fix: night mode tweaks #214

Merged
merged 3 commits into from
Jul 2, 2018
Merged

fix: night mode tweaks #214

merged 3 commits into from
Jul 2, 2018

Conversation

thatnini
Copy link

@thatnini thatnini commented Jul 1, 2018

A couple of tweaks to night mode

  • Allows the ability to see the selection of text in the editor:
    Before
    text selection before
    After
    text selection after

  • Make the test result odd rows darker:
    Before
    style multiple elements - before
    After
    style multiple elements - after

Fixes partially freeCodeCamp/freeCodeCamp/#17782

@huyenltnguyen
Copy link
Member

@thatnini @Bouncey I feel super bad about what I'm going to say. I don't want to discard someone's contributions, but we can get the vs-dark theme to work and resolve the text selection issue in the process if we, um, remove all the night styles for the code editor. :(

Here is the result on my local copy:
untitled

@Bouncey
Copy link
Member

Bouncey commented Jul 2, 2018

@huyenltnguyen No need to feel bad, if that is the fix then that is great. Have those changes been pushed up to this branch?

@thatnini
Copy link
Author

thatnini commented Jul 2, 2018

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.

@raisedadead
Copy link
Member

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

@Bouncey Bouncey merged commit 2f3c21c into freeCodeCamp:master Jul 2, 2018
@Bouncey
Copy link
Member

Bouncey commented Jul 2, 2018

Thanks @huyenltnguyen and @thatnini 👍

@huyenltnguyen
Copy link
Member

Thanks @thatnini .

@renojvarghese
Copy link

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.

@raisedadead
Copy link
Member

raisedadead commented Jul 3, 2018

@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.

Bouncey pushed a commit to Bouncey/learn that referenced this pull request Jul 27, 2018
* fix: night mode tweaks

* fix: remove custom dark theme from monaco editor
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants