sass-basis / basis

A lightweight responsive Sass/CSS framework based on flexible box.
https://sass-basis.github.io/
MIT License
144 stars 14 forks source link

Margin not exist between "_c-row--margin + _c-row _c-row--margin". #64

Closed torounit closed 8 years ago

inc2734 commented 8 years ago

どゆこと?

torounit commented 8 years ago
<div class="_c-row _c-row--margin">
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
</div>
<div class="_c-row _c-row--margin">
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
    <div class="_c-row__col _c-row__col--1-3">1/3</div>
</div>

等とした場合、._c-rowの切れ目で縦のマージンが消える。というものです。

inc2734 commented 8 years ago

なるほど! あーでもこれこういう仕様でそうしていたような気がします…。各モジュールの下マージンはフレームワーク側で定義せずに利用者側で好きに定義してねっていう。

torounit commented 8 years ago

ここら辺自分でも悩むんですけど、左右のマージンをつぶすなら、上下もつぶせよってのは筋が通ってて素敵だなーと。

それだと、ドキュメンテーションになんかコメントはした方が良いのかねぇ。