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

New Page Layout #97

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

New Page Layout #97

wants to merge 5 commits into from

Conversation

jetrotal
Copy link

I noticed that I never shared the new layout I was designing.
It was done a long ago, before I knew how work with git. So some dirty backups are inside of the files.

I need help fixing the download pages.
Didn't manage to display the player download page properly, so any commit or suggestion is welcome.

Very crude. Files were supposed to be renamed and reorganized after fixing it all.
Update main.scss
@Ghabry
Copy link
Member

Ghabry commented Mar 27, 2024

Can you share some screenshots?

@jetrotal
Copy link
Author

image
image
image
image
image

@Ghabry
Copy link
Member

Ghabry commented Mar 27, 2024

Well yeah this needs work to integrate it because - as you say - it is pretty messy with all these output files in the repo and references to cloudflare etc.

For what are you using font-awesome?

Also that gif on the main page is not acceptable because it is 1 MB in size. Our current main page is 100 KB and imo we shouldn't increase the size too much. The web is already full of bloat. There are CSS animations by now so you could use a large picture and just "scroll" it via CSS. (and if the browser is too old it will show a static so this is fine).

I think the typogruby was disabled because it corrupted some of the text. This needs checking how to workaround this as we usually want this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants