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

Upgrade The HomePage Style #409

Open
wants to merge 22 commits into
base: dev
Choose a base branch
from
Open

Upgrade The HomePage Style #409

wants to merge 22 commits into from

Conversation

kortin99
Copy link
Contributor

I have refactored the homepage code of Stream Park and upgraded the styling to make it look more modern and more enriched. The original code had several issues:

  1. Styles were not modularized or isolated, leading to potential pollution and maintenance difficulties. All styles were concentrated in the home/index.less file, making modifications challenging.
  2. The SEO was not friendly. It failed to leverage the advantages of SSR (Server-Side Rendering) for the initial page load. Incorrect module division also inadvertently generated Docusaurus routes (such as /home/feature, /home/performance), which could negatively impact the site's SEO.

My work has primarily focused on the website's homepage. I have introduced the latest atomic CSS library, Tailwind CSS, for styling, gradually replacing the previously used Bootstrap and others. This change will help reduce the build size and align our site with most contemporary websites in terms of design and performance.

localhost_3000_ (1)

@wolfboys wolfboys changed the base branch from dev to main August 18, 2024 16:20
@wolfboys wolfboys changed the base branch from main to dev August 18, 2024 16:29
@caicancai
Copy link
Member

cc @wolfboys

@wolfboys
Copy link
Member

cc @wolfboys

Hi, Calcanei. Thanks for your review. This PR will be merged when the project graduates and the official website will be updated.

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