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

Refactor to use our tamagui wrapper instead of native-base #487

Merged
merged 78 commits into from
Dec 29, 2023

Conversation

mbonete
Copy link
Collaborator

@mbonete mbonete commented Nov 11, 2023

@mbonete mbonete added the enhancement New feature or request label Nov 11, 2023
@mbonete mbonete self-assigned this Nov 11, 2023
@mbonete mbonete changed the base branch from main to andrew_testing November 11, 2023 16:05
Copy link

cloudflare-pages bot commented Nov 11, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6361623
Status:🚫  Build failed.

View logs

<HStack space={3} alignItems="center">
<Text
fontSize="lg"
fontWeight="bold"
Copy link
Owner

Choose a reason for hiding this comment

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

we might want bold still

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for the piece of feedback! I made that call while refactoring the design as I found the non-bold version closer to the apperance of the original view. I will put back the bold and prioritize keeping the HTML/design semantic over visual fidelity.

Copy link
Owner

Choose a reason for hiding this comment

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

I like your thinking! that's a good thing to be aware of

Refactor Card to use our tamagui wrapper instead of native-base
Copy link

render bot commented Nov 13, 2023

Copy link

render bot commented Nov 13, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Copy link

render bot commented Nov 13, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Copy link

render bot commented Nov 13, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

@andrew-bierman
Copy link
Owner

Possible memory leak in the RStacks in FeedCard? Trying searching with search bar and see, super laggy for me and crashes. I narrowed it down to in here maybe ? But could use second opinion

This refactor only contains the first layer of the component. The
component dependencies are still pending of refactor.
Copy link

render bot commented Nov 14, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Refactors ProfileContainer, Settings and Avatar components. Adds
headers components to packrat/ui
Copy link

render bot commented Nov 14, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Updates the Dropdown and RSelect components. As we have kept
the native version of the dropdown, I added a label to clarify
the UI. Also, removed the internal state of RSelect so all the
state is controlled from the outside.
Copy link

render bot commented Nov 15, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Copy link

render bot commented Nov 16, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

@andrew-bierman
Copy link
Owner

Dropping a comment as a reminder about memory leak / performance issue when using FeedCard. I think it's the 2 nested Stacks inside the card for some reason, but not 100%. Does not happen with native base weirdly.

@mbonete mbonete marked this pull request as draft November 17, 2023 11:26
Copy link

render bot commented Dec 27, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Copy link

render bot commented Dec 28, 2023

Your Render PR Server at https://packrat-dev-pr-487.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Copy link

render bot commented Dec 29, 2023

A deploy for your Render PR Server at https://packrat-dev-pr-487.onrender.com just failed.

View details on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

@andrew-bierman andrew-bierman marked this pull request as ready for review December 29, 2023 03:59
Copy link

render bot commented Dec 29, 2023

A deploy for your Render PR Server at https://packrat-dev-pr-487.onrender.com just failed.

View details on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

Copy link

render bot commented Dec 29, 2023

A deploy for your Render PR Server at https://packrat-dev-pr-487.onrender.com just failed.

View details on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

@andrew-bierman andrew-bierman changed the title Refactor Feed view to use our tamagui wrapper instead of native-base Refactor to use our tamagui wrapper instead of native-base Dec 29, 2023
Copy link

render bot commented Dec 29, 2023

A deploy for your Render PR Server at https://packrat-dev-pr-487.onrender.com just failed.

View details on your dashboard at https://dashboard.render.com/web/srv-cl901sqvokcc73fvjkk0.

@andrew-bierman andrew-bierman merged commit a846069 into andrew_testing Dec 29, 2023
10 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants