Change to layout to accommodate more devices below 768px

tastyigniter / ti-theme-orange

Responsive theme for TastyIgniter front-end, customizable, simple, and readable

MIT License

5 stars 21 forks source link

Change to layout to accommodate more devices below 768px #22

Closed GrantBartlett closed 3 years ago

GrantBartlett commented 3 years ago

Purpose

Devices between 576px and 768px were quite tight in terms of layout. This is only taking into account the default seeded data too. If you had long category names its likely going to cause you more issues.

Changes

Overall this only really changes the breakpoint at which the layout collapses down, replacing the $sm to $lg.

Below is a screenshot comparison, on the left is 576px and on the right is 768px.

Before (Theme as it currently stands)

Before-576-768

After (With changes applied)

After-576-768

sampoyigi commented 3 years ago

Thank you for the PR!

Shouldn't it be col-md to target devices below 768px? IIRC col-lg targets devices below 992px

Can you adjust the CartBox as well in a separate PR to the ti-ext-cart repo.

sampoyigi commented 3 years ago

@GrantBartlett Any updates?