search

Leerstandsmelder / leerstandsmelder-angular-frontend

Leerstandsmelder App - Angular.JS based frontend for Desktop & Mobile
https://leerstandsmelder.de
MIT License
4 stars 4 forks source link

scrollbars #38

Closed denjello closed 8 years ago

denjello commented 8 years ago

for some reason my local version only has one scrollbar, but live doesn't even though i pushed it. the mistake was that I used multiple / nested md-content divs...

two_scrollbars one_scrollbar

denjello commented 8 years ago

by the way, the new layout (with red on the top) should use the empty space at the top to allow the user to filter the locations...

dasantonym commented 8 years ago

i still don't like it and i think it is not fully thought through. why do we have a toolbar and then extend another toolbar from it (it is pretty much a toolshed and it can and will grow in complexity)? and why are the least important menu items on top of it (footer menu items)? and where do the search results extend (because apparently they aren't anymore, or at least in my chrome on the live site)? and how much space do you need for a configurable filter anyway (think preset list plus optional drop-down)?

hmmm