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

Adjust the website to the top #149

Closed
dperdomog opened this issue May 12, 2023 · 8 comments · Fixed by #168
Closed

Adjust the website to the top #149

dperdomog opened this issue May 12, 2023 · 8 comments · Fixed by #168
Assignees

Comments

@dperdomog
Copy link

Before
image

After
after

Is your feature request related to a problem? Please describe.
Is not considered a problem.

Describe the solution you'd like
I think the website could benefit from moving content higher since it will create more space and look better in my opinion.

Additional context
This is my first time doing this so i apologize in advance if i did something wrong. Cheers

@gdjohn4s
Copy link
Member

@dperdomog
I really like it with the tight spacing. Maybe I would add a couple of pixels between the various components, but otherwise, it's okay.
If you want, I can assign you the issue.

Thank you.

@HassanTanveer
Copy link
Contributor

@gdjohn4s Please assign this to me, if @dperdomog doesn't want it ofc

@FlavioAdamo
Copy link
Member

@HassanTanveer If he doesn't reply to us within 24h from the date of publication of the post, it will be assigned to you right away!

@HassanTanveer
Copy link
Contributor

Sounds good :)

@dperdomog
Copy link
Author

dperdomog commented May 13, 2023

Hello guys, you can assign this to @HassanTanveer , im still learning and i don't want to mess something up, im happy to make a contribution, thanks!

@gdjohn4s
Copy link
Member

@HassanTanveer any updates?

@HassanTanveer
Copy link
Contributor

Will create the PR today

@gdjohn4s
Copy link
Member

Ok, thank you

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 a pull request may close this issue.

4 participants