The Wayback Machine - https://web.archive.org/web/20201024234755/https://github.com/twbs/ratchet/pull/551
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

New "side-menus" component. #551

Open
wants to merge 1 commit into
base: master
from
Open

Conversation

@mrienstra
Copy link

@mrienstra mrienstra commented Apr 11, 2014

Based on the modals component.

See example at http://michaelrienstra.com/temporary/ratchet/side-menus.html

The current implementation relies on putting a wrapper (<div class="side-menu-siblings-wrapper">) around the side-menus siblings (<header class="bar"> & <div class="content">).

Alternatives: I have working code to handle the wrapping of siblings automatically, as well as mostly working code sans wrapper (abandoned when I realized it would take more "moving bits" to get the subtle drop shadow to animate, and the wrapper approach started to look cleaner).

Minimally tested on Android (Nexus 4) & iOS (iPhone 5).

Happy to do more testing / etc. if this component is of interest.

Attribution: I borrowed a few lines of CSS from http://mmenu.frebsite.nl/ (which is dual licensed under the MIT and GPL licenses).

Based on the `modals` component.

See example at http://michaelrienstra.com/temporary/ratchet/side-menus.html

The current implementation relies on putting a wrapper (`<div class="side-menu-siblings-wrapper">`) around the `side-menu`s siblings (`<header class="bar">` & `<div class="content">`). I have working code to handle the wrapping of siblings automatically, as well as working code that works without the wrapper (but I realized I was going to have to add more "moving bits" to get the subtle drop shadow to animate without the wrapper).

Minimally tested on Android (Nexus 4) & iOS (iPhone 5).

Happy to do more testing / etc. if this component is of interest.

Attribution: I borrowed a few lines of CSS from http://mmenu.frebsite.nl/ (which is dual licensed under the MIT and GPL licenses).
@aceport
Copy link

@aceport aceport commented Apr 11, 2014

Great!

@mrienstra
Copy link
Author

@mrienstra mrienstra commented Apr 11, 2014

@joshas
Copy link

@joshas joshas commented Apr 12, 2014

@mrienstra On Android 4.0.4 title bar doesn't move and obstructs first menu item. Also after opening there's a gap after menu, see attached screenshot.
screenshot_2014-04-12-13-01-21

@mrienstra
Copy link
Author

@mrienstra mrienstra commented Apr 13, 2014

@joshas Thanks for the heads up. Changing .bar from position: fixed to position: absolute seems to help with the title bar problem. Demo 3

Looks like Dolphin 10.3.1 (Nexus 4 running Android 4.4.2) has some problems with the animation (overshoots, then snaps back).

Haven't been able to reproduce the gap problem -- my emulator keeps crashing when I try to run 4.0 or 4.0.4. Let me know if you're in the mood to triage. :)

@cvrebert cvrebert added the feature label Apr 17, 2014
@connors connors added js labels Apr 28, 2014
@emisicka
Copy link

@emisicka emisicka commented May 20, 2014

This appears to break the modals component if they are both used on the same page. I implemented this into the "app-ios-mail" example. Even after moving the compose to a new page it appears there is some issue where it does not fire.

@mrienstra
Copy link
Author

@mrienstra mrienstra commented May 20, 2014

@emisicka: I'll take a look at that when I have time. Can you put your code somewhere I can access it?

@emisicka
Copy link

@emisicka emisicka commented May 20, 2014

@mrienstra Cool. I changed up the content with examples to recreate what I was experiencing. http://misicka.com/ratchet/

@ramilexe
Copy link

@ramilexe ramilexe commented Jun 16, 2014

Thank you, @mrienstra!

@ramilexe
Copy link

@ramilexe ramilexe commented Jun 16, 2014

@mrienstra why scroll doesn't work in menu bar?

@XhmikosR XhmikosR force-pushed the twbs:master branch from cf5785c to b227182 Aug 27, 2014
@XhmikosR
Copy link
Member

@XhmikosR XhmikosR commented Feb 9, 2015

@XhmikosR XhmikosR force-pushed the twbs:master branch from 1c7d2fa to 7d7ef88 Aug 20, 2015
@milushov
Copy link

@milushov milushov commented Nov 1, 2015

what's up? why pr still open?

@tomershvueli
Copy link

@tomershvueli tomershvueli commented Nov 28, 2016

^bump. Was debugging for a while trying to figure out why the example is working and my code is not with the same version number when I realized the PR is still open and not merged... could be nice to merge it just so Ratchet has this functionality.

@trumb1mj
Copy link

@trumb1mj trumb1mj commented Nov 28, 2016

Is there a more reliable fork of this project this could be merged into?

@tomershvueli
Copy link

@tomershvueli tomershvueli commented Nov 28, 2016

Not that I'm aware of, I'm new here... :)

@Toomavic
Copy link

@Toomavic Toomavic commented Nov 29, 2016

Great :)))) 👍

But can you add more features for menu like ::
ability to drag ,
ability to control width of menu,
make menu overlay the page not pushing the body ..

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

Successfully merging this pull request may close these issues.

None yet

You can’t perform that action at this time.