_custom.scss should be the first include

[desheikh]

Custom include should be at the top of the file. This prevents issues where for example reassigning $brand-primary would not affect any of the other variables using $brand-primary.

[bardiharborow]

This reverts #21442. Ref #21595. @mdo any idea what's going on here?

[szastupov]

Ref #21583

[mdo]

@bardiharborow The problem I was trying to solve before is that folks cannot use existing variables and mixins in the _custom.scss file if it comes first in the import order. My hope was moving it to come after the variables and mixins would solve things, but now folks can't update the value of variables that are reassigned in our _variables.scss file (e.g., $link-color using $brand-primary).

So, we're kinda screwed either way. I don't think we can have both in Sass. I think I'd rather change it back though.

Thoughts?

[judetucker]

+1 for change it back. I think it's more common for people to use it for overriding the bootstrap default vars

[Starsam80]

This PR doesn't change the files bootstrap-grid.scss and bootstrap-reboot.scss