Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Smarter handling of CSS shortcodes #74

Open
somerandomdude opened this issue Jun 2, 2013 · 7 comments
Open

Smarter handling of CSS shortcodes #74

somerandomdude opened this issue Jun 2, 2013 · 7 comments
Labels

Comments

@somerandomdude
Copy link

I'm using a lot of CSS shortcodes and it doesn't seem right that something like:

margin: 15px 0; and margin:30px 0; would be seen as having duplicates for margin-left and margin-right.

Thoughts?

@zmoazeni
Copy link
Owner

zmoazeni commented Jun 2, 2013

There's an option you can use to turn that off. --no-match-shorthand

I'm probably going to drop that feature in v2.

On Sunday, June 2, 2013, P.J. Onori wrote:

I'm using a lot of CSS shortcodes and it doesn't seem right that something
like:

margin: 15px 0; and margin:30px 0; would be seen as having duplicates for
margin-left and margin-right.

Thoughts?


Reply to this email directly or view it on GitHubhttps://github.com//issues/74
.

-Zach

@somerandomdude
Copy link
Author

Cool! Did I just miss this somewhere in the documentation or issue log? Apologies if so.

@zmoazeni
Copy link
Owner

zmoazeni commented Jun 3, 2013

It's in --help, but not really anywhere else

On Sunday, June 2, 2013, P.J. Onori wrote:

Cool! Did I just miss this somewhere in the documentation or issue log?
Apologies if so.


Reply to this email directly or view it on GitHubhttps://github.com//issues/74#issuecomment-18814988
.

-Zach

@zmoazeni
Copy link
Owner

zmoazeni commented Jun 3, 2013

I'm probably going to drop that feature in v2.

I mean the shorthand matching altogether. Not the flag to disable it.

@robme
Copy link

robme commented Mar 18, 2014

So has this been dropped? With --match-shorthand and --no-match-shorthand I can't get it to tell me that:

border-top: solid 1px #999;
and
border-top: 1px solid #999;

are the same.

@zmoazeni
Copy link
Owner

@robme Ah that's a bug. csscss shouldn't care about the order but I believe it does.

For future reference: http://stackoverflow.com/a/4077517

@zmoazeni zmoazeni reopened this Mar 18, 2014
@robme
Copy link

robme commented Mar 19, 2014

Thanks for the link. I usually always use "width style color" but the code I'm trying to refactor doesn't.

@zmoazeni zmoazeni added the bug label Aug 29, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants