这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@haxwagon
Copy link

Updating manifest to support resizeableActivity for main app and help screen.

This is also used by Samsung's DeX mode.

… screen. This is also used by Samsung's DeX mode
@haxwagon haxwagon changed the title Updating manifest to support resizeableActivity for main app and help… Updating manifest to support resizeableActivity Sep 23, 2017
@pp3345
Copy link

pp3345 commented Sep 23, 2017

This should not make any difference as resizeableActivity defaults to true already.

@haxwagon
Copy link
Author

Why doesn't it work on dex already then?

@fornwall
Copy link
Member

As @pp3345 says this attribute should already default to true:

If your app targets API level 24, but you do not specify a value for this attribute, the attribute's value defaults to true.

From https://developer.android.com/guide/topics/ui/multi-window.html.

So dex problems should come from something else.

@fornwall fornwall closed this Sep 24, 2017
@haxwagon haxwagon deleted the resizeable_activity branch October 16, 2017 20:16
@zhunsaker
Copy link

question 3 samsung developer guide

Your app might have targeting the wrong targetSDKVersion. Android:resizeableActivity is supported when the targetSDK version is greater than or equal to 24 (Android N). IF targetSDKVesion is under 24, activities are only resizable when the screenOrientation is not set as fixed. Check to make sure you do not have this code in your manifest: 

@fornwall
Copy link
Member

fornwall commented Dec 8, 2017

@twoboxen Sorry about this - you were right in that it was resizeableActivity="true" which was needed. I still don't understand why it is needed as it should be the default value when the target API is 24+ (could it be a Samsung bug perhaps), but anyway it works now.

@haxwagon
Copy link
Author

haxwagon commented Dec 8, 2017

Hey no worries. I'm just glad you got it working!

Thanks!!

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.

4 participants