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

Change runMosaic to support effects #284

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

EpicDima
Copy link
Contributor

An attempt to do #3 It works on all samples.
I haven't checked for js and native(
For jest, it seems that it would be cool to have a MosaicScope with a method for exiting, but it's more difficult to do it correctly

@JakeWharton
Copy link
Owner

Will have to look when I'm at a computer (on paternity leave until New Year).

Regarding,

For jest, it seems that it would be cool to have a MosaicScope with a method for exiting, but it's more difficult to do it correctly

I think if done properly we shouldn't need a MosaicScope anymore. Granted, we may still want one similar to how Compose Desktop provides a scope (I think with exiting functions, too).

@EpicDima EpicDima force-pushed the epicdima/run_mosaic_with_effects branch from 9a81058 to 1b6861f Compare January 8, 2024 15:13
@EpicDima EpicDima force-pushed the epicdima/run_mosaic_with_effects branch from 1b6861f to 22b1117 Compare January 19, 2024 21:28
@EpicDima EpicDima force-pushed the epicdima/run_mosaic_with_effects branch from 22b1117 to b90cf3d Compare April 20, 2024 16:14
@EpicDima EpicDima force-pushed the epicdima/run_mosaic_with_effects branch from b90cf3d to a31539c Compare May 16, 2024 09:13
@EpicDima EpicDima force-pushed the epicdima/run_mosaic_with_effects branch from a31539c to c9e05b7 Compare May 29, 2024 08:45
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.

None yet

2 participants