perf(ios): gridlayout optimizations by SeanKelly369 · Pull Request #10582 · NativeScript/NativeScript · GitHub | Latest TMZ Celebrity News & Gossip | Watch TMZ Live
Skip to content

perf(ios): gridlayout optimizations #10582

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

SeanKelly369
Copy link

PR Checklist

What is the current behavior?

The behaviour of the gridlayout is okay, but the code can be optimised. I'm trying to get the ball rolling contributing to {N}

What is the new behavior?

Optimisation of the gridlayout code for iOS.

Fixes/Implements/Closes #[Issue Number].

Copy link

cla-bot bot commented Jul 11, 2024

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: sean kelly.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot cla-bot bot added the cla: yes label Jul 11, 2024
@NathanWalker NathanWalker changed the title Origin/feat/grid layout ios optimisation perf(ios): gridlayout optimizations Jul 12, 2024
@NathanWalker
Copy link
Contributor

@SeanKelly369 possible to run some perf metrics before/after these changes?

@SeanKelly369
Copy link
Author

@NathanWalker what approach do you recommend I used to capture the performance? I could create a listview with items containing gridlayouts and calculate how much time it takes for the list to full render.

@NathanWalker
Copy link
Contributor

That would be a good baseline to start with I think @SeanKelly369 Also a page with perhaps 100 Gridlayouts with 2 labels in them (copy/paste them out) and measure the time it takes to navigate to that page with 100 gridlayouts before/after (could bump that to 200, 300 or even 400 to see broader differences to get good idea).

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

Successfully merging this pull request may close these issues.

2 participants

TMZ Celebrity News – Breaking Stories, Videos & Gossip

Looking for the latest TMZ celebrity news? You've come to the right place. From shocking Hollywood scandals to exclusive videos, TMZ delivers it all in real time.

Whether it’s a red carpet slip-up, a viral paparazzi moment, or a legal drama involving your favorite stars, TMZ news is always first to break the story. Stay in the loop with daily updates, insider tips, and jaw-dropping photos.

🎥 Watch TMZ Live

TMZ Live brings you daily celebrity news and interviews straight from the TMZ newsroom. Don’t miss a beat—watch now and see what’s trending in Hollywood.