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

Initial support for Compose/iOS #359

Merged
merged 2 commits into from
May 24, 2024
Merged

Initial support for Compose/iOS #359

merged 2 commits into from
May 24, 2024

Conversation

cedrickcooke
Copy link
Contributor

This has not been thoroughly tested for polish, but it was checked that it compiles and runs at all.
Screenshot 2024-04-24 at 1 21 55 PM

@cedrickcooke cedrickcooke requested a review from twyatt April 24, 2024 20:26
@cedrickcooke cedrickcooke added the minor Changes that should bump the MINOR version number label Apr 24, 2024
Copy link

github-actions bot commented Apr 24, 2024

Test Results

69 tests  ±0   69 ✅ ±0   1s ⏱️ -1s
12 suites ±0    0 💤 ±0 
12 files   ±0    0 ❌ ±0 

Results for commit e00dd10. ± Comparison against base commit dc7fa1c.

♻️ This comment has been updated with latest results.

@twyatt
Copy link
Member

twyatt commented May 24, 2024

I've been testing this a bit, and it has been working great.

I understand that font fallback doesn't work yet; do we need to get in changes for that before you're comfortable merging this? Or, do we just need to document that temporary shortcoming?

Perhaps any of that could come in a later PR?

Copy link
Member

@twyatt twyatt left a comment

Choose a reason for hiding this comment

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

💯

@cedrickcooke cedrickcooke marked this pull request as ready for review May 24, 2024 19:40
@cedrickcooke cedrickcooke requested a review from a team as a code owner May 24, 2024 19:40
@cedrickcooke
Copy link
Contributor Author

Not worried about font-fallback as a blocker, since desktop has the same issue

@twyatt twyatt merged commit a17eeef into main May 24, 2024
3 checks passed
@twyatt twyatt deleted the cedrickc/compose-ios branch May 24, 2024 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Changes that should bump the MINOR version number
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants