Griffith Chen
Griffith Chen
We should avoid relying on `` since it's not packaged. Might be worth looking into whether`` ([src](https://github.com/Automattic/wp-calypso/tree/trunk/packages/design-picker/src/components/theme-preview)) suits the requirements, or creating a new packaged component tailored for PA.
Moving the shuffling and limit logic to wpcom seems OK to me 👍
@daledupreez the approach LGTM, thanks for the implementation. I'll test the PR tomorrow since I'm nearing the EOD.
Tested all the flows with `isSignupFlow: true`: - ai-assembler - assembler-first-flow - design-first - free - hundred-year-plan - import-flow - import-hosted-site - link-in-bio-tld - link-in-bio - new-hosted-site-flow - newsletter -...
De-escalating this issue as it seems to only affect a12s. @miksansegundo and @arthur791004 have been actively troubleshooting this issue, see pbxlJb-5Kl-p2#comment-3728 for more context.
Same goes for other resources (e.g: `CompanyList`)
@Addison-Stavlo was this fixed by https://github.com/Automattic/wp-calypso/pull/91308?
@arthur791004 I noticed that the guided tour shows up again after page refresh, I think it's because `endTour` ([src](https://github.com/Automattic/wp-calypso/pull/91100/files#diff-9a811b118f208f223279f87118cd363f19c24c73658f19a9241c1aac82c89c22R55)) only triggers when users clicks on the Skip button, which is...
That's good to know! I'm trying the PR in via the calypso.live link, and I when click the Got it button and refresh the page, the tooltip shows up again....
The loading is faster which is fantastic, but the site switch UX seems worse as before as the previous site content will linger while the current site is still loading....