-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
feat(split-panel): split panel support for ion-nav and ion-menu #10343
Conversation
manucorporat
commented
Feb 7, 2017
•
edited
Loading
edited
I know it's a WIP but just wondering if it'll support adding custom media queries instead of just small/medium/large? |
@intellix I like the idea, probably something like:
thoughts? |
SGTM, just cause it's in the previous one :D http://ionicframework.com/docs/api/directive/exposeAsideWhen/ |
06be10f
to
d5dfe7e
Compare
@intellix it already does support custom media queries! |
3596c24
to
d3acb15
Compare
Hola, is there any way to use this features from the doc already to hide menu on login page for example or to show another menu for logged-in people ?
|
}) | ||
export class SplitPanel extends Ion implements RootNode { | ||
|
||
_rmListerner: any; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
_rmListener
:)
Hi @manucorporat. Great work with this! I'm currently adding the split pane to multiple pages within my app. Would you be willing to make ionChange an Output to follow the style of other components such as Checkbox? |
d3acb15
to
7b05d53
Compare
Found some small issues:
|
@highfieldjames what do you mean? |
07cc4a8
to
f7be9a4
Compare
@manucorporat What I mean is, at line 86, shouldn't |
@highfieldjames you are absolutely right! |
d74ab68
to
9582dcf
Compare
This looks great to me, I'm releasing a new nightly with it but I think it's good to merge. |
Revert some changes adds support split-panel support for tabs Removes .orig removes e2e.ts Removes unneeded changes in menu.ts improves stuff
27fdf6a
to
8c8377d
Compare
Is it possible to define if the main pane is shown on the right or the left? So that when the screen is too small only the left pane will be shown? Currently I have two panes
I want the main component to be shown on the left side rather then on the right to realise a detail page once the screen is big enought which will be shown on the right. |
You can change flex direction to be row reverse, that's how I solved it.
…On Tue, Mar 14, 2017, 11:07 digaus ***@***.***> wrote:
Is it possible to define if the main pane is shown on the right or the
left? So that when the screen is too small only the left pane will be shown?
Currently I have two panes
<ion-nav [root]="rootPage" #content swipeBackEnabled="false" main></ion-nav>
<ion-nav [root]="rootPage1" #content swipeBackEnabled="false"></ion-nav>
I want the main component to be shown on the left side rather then on the
right to realise a detail page once the screen is big enought which will be
shown on the right.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#10343 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAum2XQdweR-NecL_sfnui9-gRJ_FEogks5rlrtQgaJpZM4L5-eF>
.
|
Could you post your code? |
Sure:
|
Awesome, thank you! |
Nice :) |
what if there are 2 menus (left/right). Using splitpane i cant close the right menu by default. looks like menuCtl functions dont work at all when using splitpane. |
You need to manually do that. Register a click event which triggers the right menu. Does not work otherwise |
@digaus this is the problem when using splitpane, none of the functions for menu work on browser. i cant close or open using menuCtrl. |
@kodeine You also need to active the menu you want to open |
When using ion-slides with horizontal scrolling this change of b435ad6 causes a part of the next slide to be visible for +200ms when rotating from portrait to landscape as the ion-slides takes that long to resize itself. I don't have an absolute value but its long enough that after rotating the device you can see the left part of the next slide and wonder why it takes so long to resize while everything else is finished subjectively immediately @manucorporat Could you explain why this delay is needed to make ion-slides work correctly with split panes? |