Skip to content

update: Compose Multiplatform 1.7.0 What's new for upcoming beta01 #201

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

Merged
merged 18 commits into from
Sep 5, 2024

Conversation

zamulla
Copy link
Contributor

@zamulla zamulla commented Aug 21, 2024

Compiled from the changelog up to the current EAP version.

@zamulla zamulla marked this pull request as ready for review September 3, 2024 13:27
@zamulla zamulla requested a review from a team as a code owner September 3, 2024 13:27
Copy link
Contributor

@daniCsorbaJB daniCsorbaJB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, huge work 🚀 - I left a couple of comments

### Disabling minimum frame duration on iOS is mandatory

Developers often failed to notice the printed warning about the high refresh rate displays,
and users were deprived of smooth animations on their 120 Hz displays.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
and users were deprived of smooth animations on their 120 Hz displays.
resulting in users experiencing less smooth animations on their 120 Hz displays.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bit more positive, but both are fine


Developers often failed to notice the printed warning about the high refresh rate displays,
and users were deprived of smooth animations on their 120 Hz displays.
So we are enforcing this check to be strict: apps build with Compose Multiplatform will now crash
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
So we are enforcing this check to be strict: apps build with Compose Multiplatform will now crash
Therefore, we are enforcing this check to be strict: apps built with Compose Multiplatform will now crash

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or app builds

zamulla and others added 2 commits September 4, 2024 09:20
Co-authored-by: Dániel Csorba <150924204+daniCsorbaJB@users.noreply.github.com>
@zamulla zamulla requested a review from daniCsorbaJB September 4, 2024 07:33
Copy link
Contributor

@daniCsorbaJB daniCsorbaJB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cheers, LGTM! 🚀

@zamulla zamulla merged commit 2409bee into master Sep 5, 2024
@zamulla zamulla deleted the cmp-1.7.0-whats-new branch September 5, 2024 07:39
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