search

CodeForPhilly / laddr

http://codeforphilly.github.io/laddr/
MIT License
61 stars 34 forks source link

Feature - add login straight on header bar #135

Open schlos opened 8 years ago

schlos commented 8 years ago

Currently: https://qsnapnet.com/snaps/qdri7zbuaoqadcx

Expected: https://qsnapnet.com/snaps/cpmndjaveiiwl8f

All snaps: https://qsnapnet.com/folders/x6pbvqb6ciitqpv

In this way it would be easier for users to login in fewer clicks (no need to open /login/ page).

Related to this change: Since there is limited space, this maybe require that Search functionality to be displayed only to logged in users, or via menu dropdown.

themightychris commented 8 years ago

I like the idea but we have a search box up there already, and I think we'd also want a register button up there too if there's going to be a signin button. I think the search box has more value taking up the space, especially considering that Laddr sets its signin cookies to 1 year so most people won't be logging in again every time they visit the site.

Maybe a good compromise would be having the existing login button open up a modal login form and auto-focus on the username input? With the autofocus this would be the same number of clicks and page loads as embedding the fields in the toolbar

schlos commented 8 years ago

Yes, I agree, search is needed.

I like this login in modal (drop-down like) window, example: https://qsnapnet.com/snaps/wbtr0sfa1r7zm2t

Code snippit: http://www.bootply.com/60886