The size mixin is not documented in the docs.
https://github.com/twbs/bootstrap/blob/29f178d89129732d35ae812667709c64cdb28a6f/scss/mixins/_size.scss#L1-L6
Shall we:
- document this
- or remove the mixin (
widthandheightare not the most complex properties to remember)
MartijnCuppens
All 6 comments
I think we should drop this, but let's hear @mdo's thoughts too.
XhmikosR
on 12 Mar 2018
I personally find it very useful, especially in complex SCSS files.
Kopyov
on 14 Mar 2018
Not sure why anyone would want to drop this? It's not like it increases build sizes or anything. I find it cleans up the code a bit actually.
kevinruscoe
on 17 Mar 2018
Yeah, it doesn't hurt anything having it, but it does apparently need documenting.
mdo
on 18 Mar 2018
Hmm, at the moment there's no dedicated place in the documentation for the sass mixins. They are mentioned a couple of times but under their section (i.e. grid and float).
I think we would have to add a root level doc for the mixins because I'm not sure it fits anywhere else.
kevinruscoe
on 19 Mar 2018
I'm closing this.
Discussion about mixins documentation here: #25977
Once we have an overview of all the mixins, we can figure out which mixins need to be deprecated in the future.
MartijnCuppens
on 30 Mar 2018
Related issues
leomao10
路
3Comments
knownasilya
路
3Comments
kamov
路
3Comments
MrCsabaToth
路
3Comments
devdelimited
路
3Comments
Most helpful comment
Hmm, at the moment there's no dedicated place in the documentation for the sass mixins. They are mentioned a couple of times but under their section (i.e. grid and float).
I think we would have to add a root level doc for the mixins because I'm not sure it fits anywhere else.