BeastCode / VSCode-Angular-TypeScript-Snippets

Visual Studio Code TypeScript snippets (TypeScript, Html, Angular Material, Flex Layout, ngRx, RxJS & Testing) for Angular 17
https://marketplace.visualstudio.com/items?itemName=Mikael.Angular-BeastCode
MIT License
92 stars 49 forks source link

Add CSS Grid Layout Standard snippets #19

Open sociallyadequategeek opened 7 years ago

sociallyadequategeek commented 7 years ago

BeastCode, Still missing ng-cong? me too!

Now that css grid layout has been adopted by Mozilla and Chrome, would be nice to have some snips.

-MB

cgatian commented 7 years ago

@BeastCode these type of snippets feel add-on. Perhaps they are a separate package, or use a different prefix?

I guess the bottom line is what does the OP want? Examples? Css grid is dynamic and creating a useful one snip fits all is going to be quite hard.

My suggestion is, while we are refactoring the snippets for Angular 5, we place examples (if that's what the OP intended) in a new prefix, 'example--'. Given this syntax, the OP would find his grid example with 'example-css-grid-layout'

Thoughts?

cgatian commented 7 years ago

Downside to this is you don't have example snippets directly accessible from the real snips. ( Like for ng-pipe, ng-pipe-example). But, I think it actually might clean up the intellisense and provide a nice separation. Users typically know they are looking for the example when they need it. It's not something they decide when navigating the snippet intellisense.

Following this idea through, the pipe example snip would be renamed to:

example-ng-pipe

BeastCode commented 7 years ago

How should we group examples? ng-example-
m-example-
ng-example-pipe or example-ng example-m example-ng-pipe

cgatian commented 7 years ago

Doing mat- feels like a natural namespace.

So Im taking myself back, and I think we should do mat-example-<type>. But at the same time maybe you had it right the whole time!

mat-pipe-example... Gosh its pretty explicit... Im coming around to your original syntax.

cgatian commented 7 years ago

No I like what you did there. I think doing example-ng-pipe is wrong.

BeastCode commented 7 years ago

I went trough the same thoughts when I created the first example snippet. But I do see a point grouping all angular examples ng-example- but at this point there's not many examples///