BojanaMijatovic / SliderDemo

0 stars 0 forks source link

Mixini treba da imaju opciju da proslede parametre #2

Open dj-ed opened 5 years ago

dj-ed commented 5 years ago

https://github.com/BojanaMijatovic/SliderDemo/blob/19ff865e3f4050f8380a8bb9fed9140497237072/webroot/wp-content/themes/onetel-theme/src/scss/abstracts/_mixins.scss#L32

Za skoro sve mixine, u slucaju da koristis neku varijablu unutar mixin-a a mixin sadrzi vise od 1-2 linije, uvek gledaj da napravis te variabje kao parametre za taj mixin/funkciju.

Takodje prilikom imenovanja probaj da opises tacno sta je i za sta se koristi. Na primer fullWidth bi bilo bolje da je pseudoFullWidth, jer ti zapravo i pravish pseudo element a u buducnosti ces sigutno imati josh pseudo mixin-a.

@mixin pseudoFullwidth($color: $information-color) {
  content: "";
  position: absolute;
  width: 100%;
  height: 100%;
  top: 0;
  background-color: #{$color};
  z-index: -2;
} 

Imas svuda dokumentacije o tome, bitno je da zapamtish da kada se parametar koristi unutar mixina, uvek stavi #{$param}.

https://scotch.io/tutorials/how-to-use-sass-mixins