search

AmpersandJS / ampersand

Quickest way to get started with ampersand.
MIT License
812 stars 41 forks source link

Handle long lines on small screen width #104

Closed e2jk closed 9 years ago

e2jk commented 9 years ago

Adding the container class from Bootstrap to properly handled longer lines. Without this, if a line gets longer the buttons are pushed to the bottom, but the height of the list item doesn't adapt more than what the text would need. See before and after screenshots made with the default CLI-generated app, and adding just a couple of persons with a long name:

before

after

I agree this specific example (people with rather long names) seems a bit silly, but since this file is used as the base of real applications which can have lines longer than just 2 words, it makes sense to have this included in the sample template.

fyockm commented 9 years ago

+1 LGTM

kamilogorek commented 9 years ago

Same here.