Skip to content
This repository has been archived by the owner on Sep 13, 2020. It is now read-only.

iPad portrait #65

Closed
jawadrehman opened this issue Jul 8, 2015 · 19 comments
Closed

iPad portrait #65

jawadrehman opened this issue Jul 8, 2015 · 19 comments

Comments

@jawadrehman
Copy link

I am having an issue with an iPad Air in portrait mode. It doesnt take the full height of the screen.

Have also tested using the example on here.

@Kureev
Copy link
Owner

Kureev commented Jul 9, 2015

Have you tried flex: 1 for your views? Do you have your View stretched correctly without side menu?
To give you more feedback, I need screenshot and version of react-native and plugin you're using.

@jawadrehman
Copy link
Author

i have react native version 0.6, and the side menu version 0.9.0 .

I am sending pictures of how it looks like by just putting the menu item , or just putting the content.

flex : 1 is on all these components.

img_0006
img_0007
img_0008

@jawadrehman
Copy link
Author

@Kureev

@Kureev
Copy link
Owner

Kureev commented Jul 10, 2015

Wow, that's really weird, I'll take a look

@Kureev Kureev added the bug label Jul 10, 2015
@Kureev
Copy link
Owner

Kureev commented Jul 10, 2015

@alinz fyi, mb you have any thoughts so far?

@jawadrehman
Copy link
Author

i have found a hackish solution around it.

i used react-native-device module, and set the height of the container view using the height provided and it seems to work. but it would be really helpful if its sorted out.

@Kureev
Copy link
Owner

Kureev commented Jul 10, 2015

so if the issue is in the wrong height of the wrapper element - it's good, I can manage it easily. Thanks for this feedback, it'll scope my research and save my time ;)

@Kureev
Copy link
Owner

Kureev commented Jul 10, 2015

@jawadrehman You know, I took a look on implementation, and honestly I don't like it (I mean react-native-device). I prefer that kind of check made on iOS part. There is a nice gist for that https://gist.github.com/Jaybles/1323251 and I think I gonna implement that in the nearest future and plug it in as a dependency for side menu. That would help I guess.

@jawadrehman
Copy link
Author

but why would you need to set the height, isnt there a way of using flex to
do this ?

On Fri, Jul 10, 2015 at 1:04 PM, Alexey notifications@github.com wrote:

@jawadrehman https://github.com/jawadrehman You know, I took a look on
implementation, and honestly I don't like it (I mean react-native-device).
I prefer that kind of check made on iOS part. There is a nice gist for that
https://gist.github.com/Jaybles/1323251 and I think I gonna implement
that in the nearest future and plug it in as a dependency for side menu.
That would help I guess.


Reply to this email directly or view it on GitHub
#65 (comment)
.

@Kureev
Copy link
Owner

Kureev commented Jul 10, 2015

because we already use it https://github.com/Kureev/react-native-side-menu/blob/master/styles.js#L10 and it seems not working.

@Kureev
Copy link
Owner

Kureev commented Jul 10, 2015

@jawadrehman btw, are you sure your bottom menu haven't any weird paddingTop or something?

@jawadrehman
Copy link
Author

" width: deviceScreen.width,height: deviceScreen.height, " whats that about
?

On Fri, Jul 10, 2015 at 1:19 PM, Alexey notifications@github.com wrote:

because we already use it
https://github.com/Kureev/react-native-side-menu/blob/master/styles.js#L10
and it seems not working.


Reply to this email directly or view it on GitHub
#65 (comment)
.

@jawadrehman
Copy link
Author

I have tried the example code, without the tab bar and navbar and it still
has that padding left at the bottom .

On Fri, Jul 10, 2015 at 1:20 PM, Alexey notifications@github.com wrote:

@jawadrehman https://github.com/jawadrehman btw, are you sure your
bottom menu haven't any weird paddingTop or something?


Reply to this email directly or view it on GitHub
#65 (comment)
.

@Kureev
Copy link
Owner

Kureev commented Jul 10, 2015

Yeah, that's weird. Maybe you're right about deviceScreen, until scroll it should works fine, but then it breaks, right? I'll try to fix it, but I can't use flex with position: absolute

@jawadrehman
Copy link
Author

nah, it just has that padding as soon as the screen loads

On Fri, Jul 10, 2015 at 1:24 PM, Alexey notifications@github.com wrote:

Yeah, that's weird. Maybe you're right about deviceScreen, until scroll
it should works fine, but then it breaks, right?


Reply to this email directly or view it on GitHub
#65 (comment)
.

@Kureev
Copy link
Owner

Kureev commented Jul 21, 2015

You know, I tried to run my example on iPad Portrait and wasn't able to reproduce it. I'm running react-native 0.7.1 and react-native-side-menu 0.9.5. Maybe update to latest version would solve your issue?

@Kureev
Copy link
Owner

Kureev commented Aug 7, 2015

@brentvatne

@Kureev
Copy link
Owner

Kureev commented Aug 7, 2015

probably the same as #82

@Kureev
Copy link
Owner

Kureev commented Oct 26, 2015

Ok, we're not supporting RN0.6 any more, I will reopen you'll be able to experience this with latest version.

@Kureev Kureev closed this as completed Oct 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants