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

Menu has an odd interim stage on resizing #17

Open
icklecows opened this issue Jul 22, 2015 · 6 comments
Open

Menu has an odd interim stage on resizing #17

icklecows opened this issue Jul 22, 2015 · 6 comments

Comments

@icklecows
Copy link

The main navigation has an interim stage between the mobile three-line menu and the full size sidebar. It puts the whole thing full-width inbetween those two states, which takes up most of the screen.
menu-interim

@davidmce
Copy link
Contributor

Thats not great looking..
I suspect it might be somethjing to do with the way I've had to manipulate the menus. Basically, to have a mobile menu, jquery copies the menu to a new location on the page, then deletes the original menu.

Its possible that your screen resolution isn't one i tested.

@denics
Copy link

denics commented Aug 5, 2015

Hi @davidmce , if you are interested I also rewrote the menu routine to generate a menu with bootstrap tags (no more need for menu.js) and render something close to drupal admin_menu , a top black bar. If you want I can do a pull request.

@davidmce
Copy link
Contributor

davidmce commented Aug 5, 2015

@denics sounds interesting! can we see a screenshot?

@denics
Copy link

denics commented Aug 5, 2015

Ok, perhaps the red buttons are too much :) but you can customise them!

screen shot 2015-08-05 at 15 57 17

@davidmce
Copy link
Contributor

This might make more sense than the way it is at present. Would be good to have it match the installed bootstrap theme though

@denics
Copy link

denics commented Sep 14, 2015

yep, we can clean the code before posting. I will be working on this asap

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

No branches or pull requests

3 participants