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

adds stylelint config settings #590

Merged
merged 1 commit into from
Jun 8, 2016
Merged

adds stylelint config settings #590

merged 1 commit into from
Jun 8, 2016

Conversation

robertmesserle
Copy link
Contributor

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label May 31, 2016
@@ -1,5 +1,59 @@
{
"rules": {
"color-hex-case": "lower",
"color-no-invalid-hex": true,
Copy link
Member

Choose a reason for hiding this comment

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

Man, now I won't be able use to chucknorris as a color any more.

@jelbourn
Copy link
Member

LGTM, but @kara and @hansl should also chime in before we merge.

@@ -1,5 +1,59 @@
{
"rules": {
"color-hex-case": "lower",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why lower?

Copy link
Contributor Author

@robertmesserle robertmesserle May 31, 2016

Choose a reason for hiding this comment

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

Easier to type and was more common in the existing code. Also personal preference.

@robertmesserle
Copy link
Contributor Author

@jelbourn @kara @hansl Switched quotes to single and switched time to ms

}

/**
* This mixin hides an element visually.
* That means it's still accessible for screen-readers but not visible in view.
* That means it"s still accessible for screen-readers but not visible in view.
Copy link
Contributor

Choose a reason for hiding this comment

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

Missed a double quote 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, that was the one I mentioned - forgot to re-push after fixing it =x

On Tue, May 31, 2016 at 5:57 PM, Ivey Padgett notifications@github.com
wrote:

In src/core/style/_mixins.scss
#590 (comment):

}

/**

  • This mixin hides an element visually.
    • * That means it's still accessible for screen-readers but not visible in view.
    • * That means it"s still accessible for screen-readers but not visible in view.

Missed a double quote 😄


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/material2/pull/590/files/8fb026e39355a422b9e34fac2987bf85a0af9a68#r65288028,
or mute the thread
https://github.com/notifications/unsubscribe/AAi34wSiIQqHKa2z5pqu5R3bOuSki3rLks5qHNjogaJpZM4Iq1V-
.

@hansl
Copy link
Contributor

hansl commented Jun 1, 2016

I restarted browserstack.

@jelbourn
Copy link
Member

jelbourn commented Jun 1, 2016

LGTM

@iveysaur
Copy link
Contributor

iveysaur commented Jun 2, 2016

Grid list scss file needs single quotes on the import statement.

@hansl
Copy link
Contributor

hansl commented Jun 8, 2016

@robertmesserle ping.

@robertmesserle robertmesserle merged commit 4bee178 into angular:master Jun 8, 2016
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has agreed to Google's Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants