Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide unnecessary user options behind a dropdown #426

Closed
2 of 3 tasks
julianlam opened this issue Oct 21, 2013 · 5 comments
Closed
2 of 3 tasks

Hide unnecessary user options behind a dropdown #426

julianlam opened this issue Oct 21, 2013 · 5 comments
Assignees
Milestone

Comments

@julianlam
Copy link
Member

Bootstrap dropdown is fine, since Notifs use it. Opens on click, has the following options

  • Profile
  • Log out

Optionally

  • When logged out, hide "Register" and "Login" buttons behind a dropdown indicated by the fontawesome signin icon (though it's not entirely obvious, so maybe not)? @psychobunny ?
@psychobunny
Copy link
Contributor

my question is if non-techy user will be able to find register. if removed from header, a "register now" .jumbotron should probably be shown near header or possible footer, at the least.

@julianlam
Copy link
Member Author

Yeah, that's a concern of mine as well. When I first went onto Discourse, I had no idea where I was supposed to log in.

There are actually two buttons to log in on the discourse four-button layout at the top right. Confusing..

@ghost ghost assigned barisusakli Oct 21, 2013
@psychobunny
Copy link
Contributor

maybe we should make NodeBB 100% width instead of 1200, and at < 1200 implement a mobile style sidebar for additional navigation options. this would prevent IE11 from rendering NodeBB at 200% zoom too :p

either that or move menu items to the home page; have a row of buttons that take you to /recent, /unread, etc.

(thinking out loud)

barisusakli added a commit that referenced this issue Oct 21, 2013
@barisusakli
Copy link
Member

image

expanded

image

Not sure about its mobile version as drop down inside dropdown is kinda weird.

@julianlam
Copy link
Member Author

If anything, the dropdown would only be on desktop. On mobile, perhaps
there's a way to make bootstrap ignore it.
On Oct 21, 2013 6:53 PM, "Barış Soner Uşaklı" notifications@github.com
wrote:

[image: image]https://f.cloud.github.com/assets/2583394/1377251/733ea2d4-3aa3-11e3-83d2-bbeb2649953e.png

expanded

[image: image]https://f.cloud.github.com/assets/2583394/1377252/85ac08d0-3aa3-11e3-8007-3efe08b08ca9.png

Not sure about its mobile version as drop down inside dropdown is kinda
weird.


Reply to this email directly or view it on GitHubhttps://github.com//issues/426#issuecomment-26764043
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants