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

Resolving semantic colors from arbitrary packages on android #1

Merged
merged 3 commits into from
Jan 28, 2020
Merged

Resolving semantic colors from arbitrary packages on android #1

merged 3 commits into from
Jan 28, 2020

Conversation

arazabishov
Copy link

This PR adds support for semantic colors on android for arbitrary packages, as well as various ways to references colors. To showcase the change more color reference examples were added to the PlatformColorExample.

@arazabishov arazabishov changed the title Fb platformcolor android Resolving semantic colors from arbitrary packages on android Jan 27, 2020
@tom-un tom-un merged commit d9ec0ca into tom-un:fb-platformcolor Jan 28, 2020
@arazabishov arazabishov deleted the fb-platformcolor-android branch January 28, 2020 21:00
tom-un pushed a commit that referenced this pull request Sep 1, 2020
* Add ScreenshotManagerTurboModule, but it is not called.

* ...

* Update AppDelegate.mm

* Fix code due to facebook API rename

* Update AppDelegate.mm

* [Pods] Expose boost headers to consumer of RCT-Folly (#1)

* Update AppDelegate.mm

* ScreenshotManagerTurboModule gets called

* reject the promise in takeScreenshot

* ...

* Put TurboModuleRegistry.get call at the right place

* ...

* Successfully took the screenshot

* Remove unnecessary code

* Fix code review comments

* Fix code review comments

* ...

* Fix code review comments

* Fix build break

* Fix build break

* Fix build break

* Update NativeScreenshotManager.js

* Fix build break

* Fix yarn lint errors

Co-authored-by: Eloy Durán <eloy.de.enige@gmail.com>
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