Skip to content
This repository has been archived by the owner on Dec 2, 2018. It is now read-only.

Aligning vendor prefixes #79

Open
keithjgrant opened this issue May 12, 2017 · 2 comments
Open

Aligning vendor prefixes #79

keithjgrant opened this issue May 12, 2017 · 2 comments

Comments

@keithjgrant
Copy link
Collaborator

This is another option that should probably get cut. Which way to align vendor prefixes?

.dont-align {
    -webkit-user-select: none;
    -moz-user-select: none;
    -ms-user-select: none;
    user-select: none;
}

.align {
    -webkit-user-select: none;
       -moz-user-select: none;
        -ms-user-select: none;
            user-select: none;
}
@keithjgrant
Copy link
Collaborator Author

I'm in favor of not aligning (i.e. indenting normally). This avoids weird-looking "hanging" declarations beneath:

.wonky {
    -webkit-user-select: none;
       -moz-user-select: none;
        -ms-user-select: none;
            user-select: none;
    color: green;
}

As @benfrain pointed out, leave it to AutoPrefixer.

@jeddy3
Copy link

jeddy3 commented May 12, 2017

As @benfrain pointed out, leave it to AutoPrefixer.

Yes, it seems odd to add a special case to something that most people won't see. Especially when little is gained over the normal property alignment.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants