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

Update syntax for "border-radius" and "box-shadow", added vendor prefixes for calc() #2367

Merged
merged 2 commits into from Jul 11, 2016

Conversation

bensontrent
Copy link
Contributor

Simple fixes to CSS syntax. My editor was throwing errors on the CSS as border-radius:none is not a acceptable value (Should be "0"). I linted the CSS, kept all legacy css support, and added modern syntax for border-radius and box shadow.

@boneskull
Copy link
Member

@bensontrent thanks!

@@ -91,13 +91,13 @@ body {
padding: 2px 5px;
color: #fff;
-webkit-box-shadow: inset 0 1px 1px rgba(0,0,0,.2);
-moz-box-shadow: inset 0 1px 1px rgba(0,0,0,.2);
box-shadow: inset 0 1px 1px rgba(0,0,0,.2);
-moz-box-shadow: inset 0 1px 1px rgba(0,0,0,.2);
Copy link
Member

Choose a reason for hiding this comment

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

please revert whitespace changes to keep the commit clean

@boneskull
Copy link
Member

(ignore appveyor failure; known problem which is corrected in v3.0.0 branch)

@boneskull boneskull merged commit 185c0d9 into mochajs:master Jul 11, 2016
@boneskull
Copy link
Member

@bensontrent tyvm!

If you're interested in helping out, at some point we're going to want a redesign of the site...

boneskull added a commit that referenced this pull request Jul 11, 2016
* commit '185c0d902e272216232630fe4e2577268456dd9a':
  Update syntax for "border-radius" and "box-shadow", added vendor prefixes for calc() (#2367)
@montogeek
Copy link

@boneskull I want to help in the redesign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: browser browser-specific
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants