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

Compose 1.8 #1558

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Compose 1.8 #1558

wants to merge 8 commits into from

Conversation

Him188
Copy link
Member

@Him188 Him188 commented Jan 21, 2025

CMP 1.8.0-alpha02

显著变更:

  • 没有发现 bug
  • MaterialTheme.motionScheme
  • currentWindowAdaptiveInfo 变了,判断大小方式有变
  • ListDetail 框架代码变了,没有实际行为变更,但有可能支持了 predictive back
  • chips boarder 变为 outlineVariant(M3 规范更新),只影响条目详情里的 tags
  • ui test 行为更加可预测, 例如测试手指滑动屏幕调整播放进度:
    mainClock.autoAdvance = false
    root.performTouchInput {
    down(centerLeft)
    moveBy(Offset(centerX, 0f))
    }
    waitForIdle() // does nothing because autoAdvance is false
    mainClock.advanceTimeByFrame() // renders the next frame (i.e. update derivedStateOf and Text)
    mediaProgressIndicatorText.assertTextEquals("00:47 / 01:40")
    runOnUiThread {
    assertEquals(NORMAL_VISIBLE, controllerState.visibility)
    }

need #1556

@Him188 Him188 force-pushed the him188/compose-1.8 branch from e537704 to 9024e8e Compare January 22, 2025 13:14
@Him188 Him188 marked this pull request as ready for review January 22, 2025 13:14
@Him188 Him188 added this to the 4.4.0 milestone Jan 23, 2025
@Him188 Him188 enabled auto-merge (squash) January 23, 2025 02:02
@Him188 Him188 disabled auto-merge January 23, 2025 12:47
@Him188
Copy link
Member Author

Him188 commented Jan 24, 2025

ping @StageGuard

this may block all other PRs

@StageGuard
Copy link
Member

no conflict

@Him188
Copy link
Member Author

Him188 commented Jan 24, 2025

我在等你 review(你之前说要测一下)

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