-
Notifications
You must be signed in to change notification settings - Fork 78
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
NavigationView initWidth not working as expected #17
Comments
@prateekrastogi It seems that I forget to pass it to the pane style, thanks. |
@myxvisual Thanks. I was quite confused. Feel free to close this issue when fixed as I am planning to leave for a week long vacation today. |
Hi, |
@myxvisual Hi, |
@prateekrastogi Hi, sorry, some other recent private things take some time, this bug has been fixed, but not on the master branch, I will be merged in a few minutes later. Yes, if you do not like the problem of inline style, I would like to tell you about the components will automatically convert the style into the style sheet, also support className. I want to support Really thank you for your patience and support. |
@myxvisual Thanks a lot. I think i will compensate by using style components as of now. When the whole components that I want to use become clear then I will explore the possibility of refacftoring my code in to style components. Btw, styled components also support react-native https://www.styled-components.com/docs/basics#react-native. Although, I didn't looked in the depth of implications of supporting it in this library. Also, the things you are planning to support in near future sounds quite interesting especially for the domain I am using this library for. So, it would be pretty interesting to see them in future. My main issue of tilting towards syled components was initial rendering time of doc is quite slow especially in mobile browsers. I researched and found that inline styles are not as much optimised as they should be in modern browsers. Anyway, css 3d transform would be killer feature. Please do comment on the issue when navigation pane ossue is fixed and published on npm. Again, thank a lot. |
@myxvisual Also, if the re-write can be in es6, then it will a nice learning example for new-comers. You can use standard style https://github.com/standard to maintain the consistency of js code. It pretty much roots out all the dark side of js. The code also looks quite pretty without semi-colons :-) |
@prateekrastogi Hi, new version is available in npm, you can try it. |
@myxvisual Hi, I tried the new version in overlay mode. So, the initial width is working but instead of opening in compact mode , the initial width is opening in expanded mode according to initwidth parameter and on clicking expand it is further expanding according to expanded width parameter. I thought that setting initwidth parameter will increase the breadth of compact mode without showing the details that are visible in expanded mode i.e. just icon only expanded acoording to initwidth parameter. |
Here is my code |
@prateekrastogi Hi, do you mean the label text detail do not show in initMode? Just add icon width? |
@myxvisual Yep. As, of now the label text details are also showing in initMode. |
@prateekrastogi Sure, np. |
@prateekrastogi hi, I think this bug has been resolved in |
@myxvisual Thank you |
@myxvisual Hi,
Apparently, setting NavigationView initWidth property has no effect whatsoever. If my understanding is correct, then it is used for setting initial width of navigation view. Also, expanded width is working properly.
Here is the code:
<NavigationView style={{height: '100vh', margin: 10}} displayMode="overlay" autoResize={true} background="none" initWidth={100} expandedWidth={200} navigationTopNodes={navigationTopNodes} > </NavigationView>
The text was updated successfully, but these errors were encountered: