csscomb / csscomb.js

CSS coding style formatter
http://csscomb.com/
MIT License
3.29k stars 457 forks source link

Support for Polymer CSS mixins #480

Closed JosefJezek closed 5 years ago

JosefJezek commented 8 years ago

https://www.polymer-project.org/1.0/docs/devguide/styling#custom-css-mixins

.Tooltip {
        --paper-tooltip: {
          font-size: 16px;
        }
      }
Please check the validity of the block starting from line #2

1 | .Tooltip {
2*|         --paper-tooltip: {
3 |           font-size: 16px;
4 |         }

Gonzales PE version: 3.0.0-28
Syntax: css

Please ;-)

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/36741607-support-for-polymer-css-mixins?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).
tonyganch commented 8 years ago

@JosefJezek, I will appreciate a PR to parser since it should be implemented there first: https://github.com/tonyganch/gonzales-pe

JosefJezek commented 8 years ago

How to set sort-order in config for @apply mixins on top rules?

.Button--navigation {
        @apply(--layout-vertical);
        @apply(--layout-center-center);

        height: var(--header-toolbar-height);
        padding: 0;
}

After csscomb

.Button--navigation {
        height: var(--header-toolbar-height);
        padding: 0;

        @apply(--layout-vertical);
        @apply(--layout-center-center);
}
JosefJezek commented 8 years ago

Please resolve this issue... :pray:

abdonrd commented 7 years ago

+1 about this!

jdalton commented 5 years ago

Hi All!

I'm closing this because unless a PR is created it won't likely be addressed. If you feel strongly about this request I encourage you to dig in and submit a PR 😸