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

Tech tree and ctf #216

Merged
merged 10 commits into from
Mar 5, 2025
Merged

Tech tree and ctf #216

merged 10 commits into from
Mar 5, 2025

Conversation

rin-st
Copy link
Member

@rin-st rin-st commented Feb 25, 2025

note: The dashed line is in a separate component for xl+ sizes

Screenshots for different dimensions + dark theme:


Screenshot 2025-02-25 at 17 47 20
Screenshot 2025-02-25 at 17 47 03
Screenshot 2025-02-25 at 17 46 50
image

@carletex
Copy link
Collaborator

carletex commented Mar 4, 2025

Hey Rinat, this is looking pretty good!

I swapped the PNGs with SVGs and tweaked the title / description a bit (getting some confirmation on this)

A couple of things:

  1. On a desktop screen the text and image are too far away

image

It should be a little more like this, it's ok if there is empty bg on the sides (Maybe we can limit the width)

image

  1. Is it possible to make the background infinite? As in the Join the BG section

This is not a big deal, but let's do it if it's an easy tweak.

image

@carletex carletex force-pushed the tech-tree-and-ctf branch from 540a2e7 to c4c4d3b Compare March 4, 2025 10:59
@rin-st
Copy link
Member Author

rin-st commented Mar 4, 2025

Thanks @carletex ! Updated

image

@rin-st
Copy link
Member Author

rin-st commented Mar 5, 2025

Changed svg and hardcoded padding for techTree
image

Copy link
Collaborator

@carletex carletex left a comment

Choose a reason for hiding this comment

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

Good stuff Rinat!!!

Pushed some little RWD tweak + changed the tech tree link

Thank you

@carletex carletex merged commit 096e577 into master Mar 5, 2025
1 check failed
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.

2 participants