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

suniti heading font changes to ariel, h2 color changed to black. #119

Closed
wants to merge 1 commit into from

Conversation

sunitijain111
Copy link
Contributor

@sunitijain111 sunitijain111 commented Apr 18, 2020

Closes #95

hey! in public/ css-styles i made a couple of changes.
I have change heading fonts to ARIEL and h2 colour to black.

Copy link
Member

@lisa761 lisa761 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sunitijain111, hey good work!
Although I am not too sure about Comic sans instead of the current font. We can wait for what @soumyaa1804 thinks.

}

h2 {
color: #313338;
color: #ffffff;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#ffffff can also be written as #fff, it means the same thing. Plus #fff is actually white and not black. But I don't see anything changing because of this?

@sunitijain111
Copy link
Contributor Author

sunitijain111 commented Apr 20, 2020 via email

@lisa761
Copy link
Member

lisa761 commented Apr 22, 2020

So, it'd be great if you changed the color back to what it was before, if it doesn't work :)

@@ -0,0 +1,45 @@
0 info it worked if it ends with ok
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You seem to have added this extra npm-debug.log file in your commit. Make sure you remove this, as it would not be required. To see what I am talking about, you can click on that Files Changed button at the top of your PR.
When you are adding files to a commit, you can run git status to see how many files have changed or been staged for the commit. If you changed multiple files, but only want to commit one, say File1.md. You can do that by running add File.md instead of add .. The . in the latter means that it adds all the files that have changed.

@sunitijain111
Copy link
Contributor Author

sunitijain111 commented Apr 23, 2020 via email

@lisa761
Copy link
Member

lisa761 commented Apr 23, 2020

Did you check with your master? Or like I explained in the most recent review to see how you can check the changes you have made? Because it was #313338 and not #ffffff. Please make sure you read all the reviews correctly, and acknowledge them.

@lisa761
Copy link
Member

lisa761 commented Jun 25, 2020

Closing this since it's been inactive for a while. Feel free to reopen if you'd still like to get this merged :)

@lisa761 lisa761 closed this Jun 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

for task 5, changes
2 participants