-
Notifications
You must be signed in to change notification settings - Fork 135
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
Create text variant defaults is not working. #157
Comments
Hey @alphonse92, could you try the following changes locally and see if they fix this issue? https://github.com/Shopify/restyle/pull/152/files |
I wasn't able to try your solution, however, I can confirm I had a similar issue as the above poster, and downgrading to 2.0.0 solved it for me. When I upgraded to 2.1.0, the app would immediately crash if any Text components were on screen. The error code I got was:
|
I'm having the same issue. running restyle 2.1.0. Going to try to downgrade to 2.0.0 |
Can confirm I'm still having the same issue and this suggestion worked then and still works now. |
@sbalay Yes it is working, can we have this fix in the release branch? |
same here. |
This is still broken, and in addition, #35 is now appearing in new apps as well, and both are resolved by downgrading to 2.0.0. It's a 50/50 of which of these issues appear in a new app I'm working on. At this point, I can only assume 2.1.0 is broken and has been for some time. Luckily 2.0.0 is fully stable, as I'm using this in 5 different apps and have been for a few months, so if anyone is using this library, make sure you |
Hey @alphonse92 and @alexanderblackh, I'm unable to reproduce locally. Are you still seeing this issue? |
I have run into this issue using version 2.4.2 |
Same. I get |
Also having that problem in 2.4.2 |
Hey y'all, sorry for ghosting on this thread, I gave up sometime ago and found downgrading for my prior project provided what I needed. However, on a new project this re-appeared again that used an old theme.ts that outright lacked a |
Restyle components do not use the default styles unless you specify it
variant="defaults"
in the props.My component looks like:
The theme object variant looks like:
Video showing the variant only working when I set
variant="defaults"
Screen.Recording.2022-05-05.at.3.32.55.PM.mov
Version
More info:
I created a wraper component to inspect the style object passed down by the restyle wraper. As you can see the first two outputs are printed when
variant is undefined
but the next two outputs shows are printed whenvariant="defaults"
The text was updated successfully, but these errors were encountered: