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

[3.0] Fix issues with mobile website #2416

Merged
merged 7 commits into from
Feb 2, 2025

Conversation

joskuijpers
Copy link
Contributor

@joskuijpers joskuijpers commented Jan 25, 2025

Summary of the PR

This fixes a bunch of issues with the styling on the new website, especially for mobile.
Also cleaned things up a bit on the icons.

I have also updated the used packages to the ones used by a new Docosaurus project.
Included TailwindCSS as it is much easier to build mobile/web sites with it than with raw CSS and the mess React makes of it.

Further Comments

For developing this I needed to disable cleanup of nuke website so that npm start worked. This should get a bit nicer dev experience. Maybe by putting the generated files in a folder source/Website/.generated and keeping it there with a gitignore.

@joskuijpers joskuijpers requested a review from a team as a code owner January 25, 2025 20:57
@joskuijpers
Copy link
Contributor Author

@dotnet-policy-service agree

@Perksey Perksey enabled auto-merge (squash) February 2, 2025 16:29
@Perksey Perksey disabled auto-merge February 2, 2025 16:59
@Perksey Perksey merged commit e9da1e0 into dotnet:develop/3.0 Feb 2, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants