Skip to content

[DO NOT MERGE] Brian doyle/update homepage #1572

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

[DO NOT MERGE] Brian doyle/update homepage #1572

wants to merge 11 commits into from

Conversation

@briandoyle81
Copy link
Contributor

@briandoyle81 briandoyle81 commented Nov 12, 2025

No description provided.

@vercel
Copy link

vercel bot commented Nov 12, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Nov 20, 2025 10:01pm

briandoyle81
briandoyle81 commented Nov 12, 2025
View reviewed changes
src/ui/design-system/src/lib/Pages/HomePage/GridData/BuildGridData.tsx
variant: 'horizontal',
href: '../blockchain-development-tutorials/cross-vm-apps',
},
// {
Copy link
Contributor Author

@briandoyle81 briandoyle81 Nov 12, 2025

Choose a reason for hiding this comment

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

@chasefleming I'll clean this up before committing. Leaving for now in case someone wants it back.

@chasefleming
Copy link
Member

chasefleming commented Nov 12, 2025

Looks good, a couple notes:

  • I like the "Why Flow" section
  • Nit: cursor on "Run Script" should be a pointer
  • Script runner section looks nice
  • When the script has already ran on load I don't realize it, then I click and it does nothing. Maybe we could make the result show fetching or something when clicked? And refetch?
  • Edit button should be a cursor pointer as well
  • Can React SDK be in the tools section? It's probably one of the most important.
  • I don't think we should link to Hackathons and Events. That page is sparse and I'm not sure there a lot of events going on right now. We are also linking to a calendar in the card before. Maybe a link to X?
  • Should there be an image in the empty space at the top right?

cshannon1218
cshannon1218 reviewed Nov 13, 2025
View reviewed changes
@briandoyle81 briandoyle81 changed the title Brian doyle/update homepage [DO NOT MERGE] Brian doyle/update homepage Nov 13, 2025
@briandoyle81 briandoyle81 marked this pull request as draft November 13, 2025 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cshannon1218 cshannon1218 cshannon1218 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@briandoyle81 @chasefleming @cshannon1218