Sitelet https://github.com/KeepSafe/TapTargetView/pull/187
Skip to content

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

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

zawadz88 wants to merge 1 commit into
KeepSafe: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

xiphirx commented Jul 23, 2017

Copy link
Copy Markdown
Collaborator

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

xiphirx commented Jul 5, 2018

Copy link
Copy Markdown
Collaborator

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