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.

Sign up for GitHub

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

Jump to bottom

refactor: restyle #32

Merged
merged 5 commits into from
Jul 6, 2023
Merged

refactor: restyle #32

merged 5 commits into from
Jul 6, 2023

Conversation

razgraf
Copy link
Member

@razgraf razgraf commented Jul 4, 2023 •
edited by PaulRBerg
Loading

Description

Closes #2.

Preview

Dark (minimal color swaps)

Homepage Page
Screenshot 2023-07-04 at 22 13 37 Screenshot 2023-07-04 at 22 15 26

Light (minimal color swaps)

Homepage Page
Screenshot 2023-07-04 at 22 27 09 Screenshot 2023-07-04 at 22 28 02

@razgraf razgraf changed the title refactor: restyle [WIP] refactor: restyle Jul 4, 2023
@razgraf razgraf changed the title [WIP] refactor: restyle refactor: restyle Jul 4, 2023
@razgraf razgraf requested a review from PaulRBerg July 4, 2023 19:29
@razgraf razgraf mentioned this pull request Jul 4, 2023
Merged
@maxdesalle
Copy link
Member

This looks so good! 🤩

@razgraf
Copy link
Member Author

razgraf commented Jul 6, 2023 •
edited
Loading

Tasks

  • The font weight used for the content needs to be lowered by 100 - for some reason post-build the css doesn't apply as intended.
  • Rebase

PaulRBerg
PaulRBerg approved these changes Jul 6, 2023
View reviewed changes
Copy link
Member

@PaulRBerg PaulRBerg left a comment

Choose a reason for hiding this comment

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

The new theme and home page look lit!

I've left a few comments as feedback + created two GitHub issues to track two other smol design requests.

I've also removed the index.tsx files because they make the code base harder to search in VSCode.

.prettierrc.yml Outdated Show resolved Hide resolved
tsconfig.json Show resolved Hide resolved
src/components/molecules/Feature/index.tsx Outdated Show resolved Hide resolved
docusaurus.config.js Outdated Show resolved Hide resolved
src/css/custom.css Outdated Show resolved Hide resolved
src/css/sablier.css Show resolved Hide resolved
@razgraf
fix: type annotation not recognized by typescript
d4b29c4
@razgraf
fix: tsconfig warning about module compatibility with resolutions
009b534
@razgraf
feat: redesign the home page
865e33d 
style: modify colors for the nav and footer
fix: remove duplicate redirect from docusaurus config
feat: sort atomic components and remove unused ones
@razgraf
style: add minimal support for light mode
98f53a7
@razgraf
fix: code review
45db51c 
chore: remap lint and prettier scrips, include check and fix scripts
ci: replace lint with check script
@razgraf razgraf merged commit 49980f5 into main Jul 6, 2023
@razgraf razgraf deleted the refactor/restyle branch July 6, 2023 21:23
@PaulRBerg
Copy link
Member

@razgraf you might have accidentally overridden my commit (you pushed with --force). My commit 5aa064f is no longer in the history of this branch.

I will create a separate PR to re-implement my changes.

@PaulRBerg PaulRBerg mentioned this pull request Jul 7, 2023
Merged
@razgraf
Copy link
Member Author

razgraf commented Jul 7, 2023

@razgraf you might have accidentally overridden my commit (you pushed with --force). My commit 5aa064f is no longer in the history of this branch.

I will create a separate PR to re-implement my changes.

So sorry about this, I didn't realize I wasn't following the latest version locally (weirdly enough I don't remember git pull screaming about it when I ran it). Thank you for re-implementing the changes!

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

@PaulRBerg PaulRBerg PaulRBerg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Prettify the website
3 participants
@razgraf @maxdesalle @PaulRBerg