【追加要望】CSS - Githubissues

lincwell / frontend-coding-guideline

Linc'well Frontend Coding Rules

1 stars 0 forks source link

Closed diverstage closed 2 years ago

diverstage commented 2 years ago

idセレクタ、タイプセレクタへのスタイル指定はしない（classセレクタのみ）
- スタイルがタイプに依存することで、保守性が下がるため
- ulやolに対してのliや、dlに対してのdtやddなどは対象外（構造としてタイプが決まってしまっていて変更になる可能性が少ないため）

// Good
.header {
  / * some style */
}

// Bad
header {
  / * some style */
}

// Bad
#header {
  / * some style */
}

// Good
.hoge-fuga {
  / * some style */
}

// Bad
.hoge .fuga {
  / * some style */
}

// Good
.example {
 margin-top: 0;
}

// Bad
.example {
 margin-top: 0px;
}

to-na commented 2 years ago

内容良いと思います。文言もこのままで問題なさそうなので、html-and-css/README.md に追記する形でPR作ってOKです。

diverstage commented 2 years ago

@to-na

Git: Permission to lincwell/frontend-coding-guideline.git denied to diverstage.

プッシュしようとするとこの様に出るので、閲覧はできるのですが、プッシュする権限は私にはない感じですかね🤔

to-na commented 2 years ago

Push しようとしているのは main ブランチですか？通常 main に直接 Push できないようにしているので、その場合はその影響だと思われます。

diverstage commented 2 years ago

@to-na

Push しようとしているのは main ブランチですか？

いえ、さすがにmainに直接Pushはしてなくて feature/add_css_guide_20220207 ってブランチ切ってプッシュしてプルリクを作ろうとしてましたね。

Clone は普通にできたので何でなんかなと思いまして🤔

私の方の設定をもう一度確認してみたいと思います🙇‍♂️

to-na commented 2 years ago

@diverstage こちら direct access の権限がなかったためだと思われます 🙇 invite しようとしましたが seats が少ないので自分の方で追記しておきます！

diverstage commented 2 years ago

@diverstage こちら direct access の権限がなかったためだと思われます 🙇 invite しようとしましたが seats が少ないので自分の方で追記しておきます！

@to-na ありがとうございます！よろしくお願いいたします🙇‍♂️

to-na commented 2 years ago