Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

Fixes to make ExpoPixi.Signature work on web #148

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

guyzmo
Copy link

@guyzmo guyzmo commented Oct 23, 2020

  1. disables the getContextAttributes from context on web platform cf Issue when running Signature on Expo Web #147
  2. changes the ref assignment for GLView. There's an issue opened on expo-gl, because I believe it should be fixed there, not here. cf ExpoGL on the Web: crash because of ref prop on GLView expo#10774.
  3. Fixes the takeSnapshotAsync() call for the web platform by using the static call. I'm not sure why the local utils one is actually failing to call the bound one.

About 3. I did not make an issue, because even though the takeSnapshotAsync call works, it only saves blank images. I need to figure out why (maybe I'm giving it a wrong context? 🤔 )

This could be three distinct one commit PR, though I believe they all belong together as it's achieving one goal: make Signature work on the web :)

@guyzmo
Copy link
Author

guyzmo commented Sep 2, 2021

Hello 👋 It's been almost a year since I pushed that PR and that I'm running a fork of expo-pixi. Is there someone left to maintain this code base ? 🤔

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant