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

Fix submenu for small width (#62) #63

Merged
merged 1 commit into from
Aug 22, 2013
Merged

Fix submenu for small width (#62) #63

merged 1 commit into from
Aug 22, 2013

Conversation

muan
Copy link
Member

@muan muan commented Aug 22, 2013

Issues:

  1. In desktop, the hovering drops the right-pointing triangles to the middle (should be top);
  2. Touch-only devices cannot toggle the expansion of sub-menu items at all:

Both are fixed. 2 was cause by <a> not having href attribute.

As for the bootstrap suggestion, I don't see one easily enough to implement. I'd tend to think if there is, it would be included in the official docs. As I previously mentioned, it has been discussed in bootstrap about the reason for dropping support of submenus, and also a suggested method(which is pretty much what we have here.)

So.. still...
cooool

@audreyt
Copy link
Member

audreyt commented Aug 22, 2013

Wow, you're fast!

Next time feel free to directly commit to the master branch -- you're a committer after all. 👍

audreyt added a commit that referenced this pull request Aug 22, 2013
Fix submenu for small width (#62)
@audreyt audreyt merged commit 3cfb9e8 into master Aug 22, 2013
@audreyt
Copy link
Member

audreyt commented Aug 22, 2013

Oh, now I see this very helpful "Pull request successfully merged and closed — Delete branch" UI for the very first time! 🆒

That makes the feature-branch workflow much easier, so nvm what I just said about the master branch. 😊

@audreyt audreyt deleted the submenu-fixes branch August 22, 2013 21:54
@muan
Copy link
Member Author

muan commented Aug 22, 2013

I am all in the :octocat: way. 😜

audreyt added a commit that referenced this pull request Jan 18, 2014
Fix submenu for small width (#62)
audreyt added a commit that referenced this pull request Jan 23, 2014
Fix submenu for small width (#62)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants