Skip to content

Fix for issue #91, i.e. adding multi window support#187

Closed
zawadz88 wants to merge 1 commit intoKeepSafe:masterfrom
zawadz88:multiwindow-support
Closed

Fix for issue #91, i.e. adding multi window support#187
zawadz88 wants to merge 1 commit intoKeepSafe:masterfrom
zawadz88:multiwindow-support

Conversation

@zawadz88
Copy link
Copy Markdown

Hi,
This is a fix for #91.

Before the changes the following was happening:

screenshot_1497348039
screenshot_1497348057

These changes work for me in the sample app and in the app I'm working on, however I changed the part below the FLAG_LAYOUT_NO_LIMITS comment which I couldn't find how to reproduce so I'm not sure if this will work in this scenario.

@xiphirx
Copy link
Copy Markdown
Collaborator

xiphirx commented Jul 23, 2017

Hmm, yeah this was sort of what I had before I changed it to accommodate FLAG_LAYOUT_NO_LIMITS. You're correct, this will break compatibility with FLAG_LAYOUT_NO_LIMITS, so I'll need to think about how to tackle this properly.

@philippb
Copy link
Copy Markdown
Contributor

@xiphirx how are things going on your end. You think you have a chance to follow up on this? Would be great.

@philippb
Copy link
Copy Markdown
Contributor

@zawadz88 thank you for submitting this PR and bringing its status to Engin attention at DriodCon in Berlin.

@xiphirx
Copy link
Copy Markdown
Collaborator

xiphirx commented Jul 5, 2018

Fixed in #300

@xiphirx xiphirx closed this Jul 5, 2018
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

Successfully merging this pull request may close these issues.

3 participants