-
Notifications
You must be signed in to change notification settings - Fork 289
Revised formatting of some snippets. #478
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
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -204,23 +204,6 @@ else if (parentConfiguration.screenWidthDp >= 840) { | |
| ruleController.addRule(activityRule); | ||
| // [END android_activity_embedding_addRuleActivityRule_java] | ||
|
|
||
| // [START android_activity_embedding_splitAttributesBuilder_java] | ||
| SplitAttributes.Builder _splitAttributesBuilder = new SplitAttributes.Builder() | ||
| .setSplitType(SplitAttributes.SplitType.ratio(0.33f)) | ||
| .setLayoutDirection(SplitAttributes.LayoutDirection.LEFT_TO_RIGHT); | ||
|
|
||
| if (WindowSdkExtensions.getInstance().getExtensionVersion() >= 6) { | ||
| _splitAttributesBuilder.setDividerAttributes( | ||
| new DividerAttributes.DraggableDividerAttributes.Builder() | ||
| .setColor(ContextCompat.getColor(this, R.color.divider_color)) | ||
| .setWidthDp(4) | ||
| .setDragRange(DividerAttributes.DragRange.DRAG_RANGE_SYSTEM_DEFAULT) | ||
| .build() | ||
| ); | ||
| } | ||
| SplitAttributes _splitAttributes = _splitAttributesBuilder.build(); | ||
| // [END android_activity_embedding_splitAttributesBuilder_java] | ||
|
|
||
| } | ||
|
|
||
|
|
||
|
|
@@ -336,6 +319,23 @@ protected void onCreate(@Nullable Bundle savedInstanceState) { | |
| super.onCreate(savedInstanceState); | ||
| setContentView(R.layout.activity_main); | ||
|
|
||
| // [START android_activity_embedding_splitAttributesBuilder_java] | ||
| SplitAttributes.Builder splitAttributesBuilder = new SplitAttributes.Builder() | ||
| .setSplitType(SplitAttributes.SplitType.ratio(0.33f)) | ||
| .setLayoutDirection(SplitAttributes.LayoutDirection.LEFT_TO_RIGHT); | ||
|
|
||
| if (WindowSdkExtensions.getInstance().getExtensionVersion() >= 6) { | ||
|
||
| splitAttributesBuilder.setDividerAttributes( | ||
| new DividerAttributes.DraggableDividerAttributes.Builder() | ||
| .setColor(ContextCompat.getColor(this, R.color.divider_color)) | ||
| .setWidthDp(4) | ||
| .setDragRange(DividerAttributes.DragRange.DRAG_RANGE_SYSTEM_DEFAULT) | ||
| .build() | ||
| ); | ||
| } | ||
| SplitAttributes _splitAttributes = splitAttributesBuilder.build(); | ||
| // [END android_activity_embedding_splitAttributesBuilder_java] | ||
|
|
||
| // [START android_activity_embedding_pinButton_java] | ||
| Button pinButton = findViewById(R.id.pinButton); | ||
| pinButton.setOnClickListener( (view) -> { | ||
|
|
@@ -349,16 +349,15 @@ protected void onCreate(@Nullable Bundle savedInstanceState) { | |
| .setDefaultSplitAttributes(splitAttributes) | ||
| .build(); | ||
|
|
||
| SplitController.getInstance( | ||
| getApplicationContext()).pinTopActivityStack(getTaskId(), | ||
| pinSplitRule); | ||
| SplitController.getInstance(getApplicationContext()) | ||
| .pinTopActivityStack(getTaskId(), pinSplitRule); | ||
| }); | ||
| // [END android_activity_embedding_pinButton_java] | ||
|
|
||
| // [START android_activity_embedding_getSplitSupportStatus_java] | ||
| if (SplitController.getInstance(this).getSplitSupportStatus() == | ||
| SplitController.SplitSupportStatus.SPLIT_AVAILABLE) { | ||
| // Device supports split activity features. | ||
| SplitController.SplitSupportStatus.SPLIT_AVAILABLE) { | ||
| // Device supports split activity features. | ||
| } | ||
| // [END android_activity_embedding_getSplitSupportStatus_java] | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to note, we generally create functions that separate out each snippet logically, otherwise reading the file in its entirety is a bit difficult if they are all in the same function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reorganized the files to group related snippets.