Skip to content

Comments

Enhance landing page#39

Merged
carletex merged 9 commits intomainfrom
feat/more-stuff-in-homepage
Dec 22, 2025
Merged

Enhance landing page#39
carletex merged 9 commits intomainfrom
feat/more-stuff-in-homepage

Conversation

@portdeveloper
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Dec 18, 2025

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

Project Deployment Review Updated (UTC)
ctf.buidlguidl.com Ready Ready Preview, Comment Dec 22, 2025 9:05pm
1 Skipped Deployment
Project Deployment Review Updated (UTC)
ctf-devcon Ignored Ignored Dec 22, 2025 9:05pm

Copy link
Member

@damianmarti damianmarti left a comment

Choose a reason for hiding this comment

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

This looks great @portdeveloper Thanks!!

Added a comment to avoid repeating code.

I think the tooltip can look better. Not sure how, maybe changing the background color, so it has more contrast.

Copy link
Member

@damianmarti damianmarti left a comment

Choose a reason for hiding this comment

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

Thanks for the refactor!

And the tooltip looks much better now!

@carletex what do you think?

@carletex
Copy link
Member

Looking pretty good!! Thanks Port <3

Some things that I see.

  1. Let's add a bit more space between the start and the begining of the columns.
image

At least something like this
image

  1. Let's place the tooltip in another position. Usually you hover the challenges from top to bottom. It's a bit weird to just run over the tooltip when hovering (and also hides the next item).
image

Maybe on the right?

  1. I think we could have better descriptions for the tootlip. I think we can be more specific about what you are going to learn / need to apply. (we can also remove the "mint your flag", "interact with challenge X", etc). IF not, it feels a bit vague?

Some examples:
"Mint your flag in this unverified contract." => "Analyze the bytecode of a unverified contract" or just "bytecode on unverified contract"
"Mint your flag using the right signature." => "Craft a signature of a known PK" or just "Crafting signatures"

Really open on this one, what do you think? We can also update the descriptions in another PRs.

cc @portdeveloper @damianmarti

@damianmarti
Copy link
Member

Looking pretty good!! Thanks Port <3

Some things that I see.

  1. Let's add a bit more space between the start and the begining of the columns.
image At least something like this image
  1. Let's place the tooltip in another position. Usually you hover the challenges from top to bottom. It's a bit weird to just run over the tooltip when hovering (and also hides the next item).
image Maybe on the right?
  1. I think we could have better descriptions for the tootlip. I think we can be more specific about what you are going to learn / need to apply. (we can also remove the "mint your flag", "interact with challenge X", etc). IF not, it feels a bit vague?

Some examples: "Mint your flag in this unverified contract." => "Analyze the bytecode of a unverified contract" or just "bytecode on unverified contract" "Mint your flag using the right signature." => "Craft a signature of a known PK" or just "Crafting signatures"

Really open on this one, what do you think? We can also update the descriptions in another PRs.

cc @portdeveloper @damianmarti

I'm not sure. I think that the current vague descriptions are intended, like part of the game, where you need to find out what you have to do. But I understand that if we have better descriptions, the users can read these and search for the type of challenge they want to do. Both options are ok for me.

@portdeveloper
Copy link
Member Author

Thanks for the comments! Implemented 1 and 2 but waiting on 3.

@carletex
Copy link
Member

Thanks! After tinkering a bit, moved the tooltip on the left.

Regarding the tooltip descriptions:

  • I think we should have better ones. If not, they don't add much.

In any case, let's merge this for now!

@carletex carletex merged commit 651391b into main Dec 22, 2025
4 checks passed
@carletex carletex deleted the feat/more-stuff-in-homepage branch December 22, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants