Fixes: CSS column alignment

[machikoyasuda]

closes #1548

• Desktop: Container width for most* pages is now col-lg-8, not col-lg-10. Index page, Agency Index page and Enrollment Success is an outlier and is not part of this. No changes necessary for 404/400/500 pages. Also one page has a col-lg-6 H1 wrapper container instead. Because some pages are actually different from others, I recommend not codifying the col-lg-8 width into the base.html itself.
• Desktop: Media Item grid gap is now lower to 24px.
• Mobile: Should have no changes

Screenshots

[thekaveman]

This looks good.

Just wondering about:

I recommend not codifying the col-lg-8 width into the base.html itself.

Would that come in a later refactor?

[machikoyasuda]

@thekaveman

_Just wondering about:

I recommend not codifying the col-lg-8 width into the base.html itself.

Would that come in a later refactor?_

No - because not all pages have col-lg-8 width by default. So I'd rather not put it in.. because it's not on all pages. Help, Enrollment Success, Log Out, Agency Index, Index all are different. If "base" is truly to be "base", it should serve all pages.

[thekaveman]

@thekaveman

_Just wondering about:

I recommend not codifying the col-lg-8 width into the base.html itself.

Would that come in a later refactor?_

No - because not all pages have col-lg-8 width by default. So I'd rather not put it in.. because it's not on all pages. Help, Enrollment Success, Log Out, Agency Index, Index all are different. If "base" is truly to be "base", it should serve all pages.

@machikoyasuda that was my question, it's still in this PR -- will removing it be in a separate PR? (I guess so)

[machikoyasuda]

@thekaveman Oh sorry I think I figured out your question.

https://github.com/cal-itp/benefits/pull/1550/files#diff-e2ae5f60bb472effafac24b079fcbb1ffd09dcc5d8df2d22cab07417d222f925R72-R87

I DO recommend col-lg-8 be in headline and explanatory-text -- b/c all pages do use this, as this PR reflects. But I do not recommend col-lg-8 to be coded into base.html for the inner-content portion, cos that's where it differs for the other pages.