Skip to content
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

Prototype with pager #721

Open
wants to merge 32 commits into
base: 1.x
Choose a base branch
from

Conversation

vladcipariu91
Copy link
Collaborator

Won't merge, just showcasing the pager

KovalevAndrey and others added 30 commits April 22, 2024 14:47
- Moved LocalMovableContentMap to Children composable and added a parameter to it.
- Added support for more movableContent API variants
@@ -0,0 +1,4 @@
package com.bumble.appyx.sandbox.client.sharedelement.with_pager

Check warning

Code scanning / detekt

Package names should match the naming convention set in the configuration. Warning

Package name should match the pattern: [a-z]+(.[a-z][A-Za-z0-9]*)*
Comment on lines +3 to +4
class ShareElementDetailsNode {
}

Check warning

Code scanning / detekt

Empty block of code detected. As they serve no purpose they should be removed. Warning

The class or object ShareElementDetailsNode is empty.
package com.bumble.appyx.sandbox.client.sharedelement.with_pager

class ShareElementDetailsNode {
}

Check warning

Code scanning / detekt

Checks whether files end with a line separator. Warning

The file /home/runner/work/appyx/appyx/samples/sandbox/src/main/kotlin/com/bumble/appyx/sandbox/client/sharedelement/with_pager/ShareElementDetailsNode.kt is not ending with a new line.
@@ -0,0 +1,4 @@
package com.bumble.appyx.sandbox.client.sharedelement.with_pager

Check warning

Code scanning / detekt

Package names should match the naming convention set in the configuration. Warning

Package name should match the pattern: [a-z]+(.[a-z][A-Za-z0-9]*)*
Comment on lines +3 to +4
class SharedElementPagerNode {
}

Check warning

Code scanning / detekt

Empty block of code detected. As they serve no purpose they should be removed. Warning

The class or object SharedElementPagerNode is empty.
package com.bumble.appyx.sandbox.client.sharedelement.with_pager

class SharedElementPagerNode {
}

Check warning

Code scanning / detekt

Checks whether files end with a line separator. Warning

The file /home/runner/work/appyx/appyx/samples/sandbox/src/main/kotlin/com/bumble/appyx/sandbox/client/sharedelement/with_pager/SharedElementPagerNode.kt is not ending with a new line.
@@ -0,0 +1,146 @@
package com.bumble.appyx.sandbox.client.sharedelement.with_pager

Check warning

Code scanning / detekt

Package names should match the naming convention set in the configuration. Warning

Package name should match the pattern: [a-z]+(.[a-z][A-Za-z0-9]*)*
}

@OptIn(ExperimentalSharedTransitionApi::class)
override fun resolve(navTarget: NavTarget, buildContext: BuildContext): Node {

Check warning

Code scanning / detekt

One method should have one responsibility. Long methods tend to handle many things at once. Prefer smaller methods to make them easier to understand. Warning

The function resolve is too long (67). The maximum length is 60.
transitionHandler = rememberBackstackFader(transitionSpec = { tween(300) })
)
}
}

Check warning

Code scanning / detekt

Checks whether files end with a line separator. Warning

The file /home/runner/work/appyx/appyx/samples/sandbox/src/main/kotlin/com/bumble/appyx/sandbox/client/sharedelement/with_pager/SharedElementPagerParentNode.kt is not ending with a new line.
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.

2 participants