Skip to content

Conversation

@Harsh-br0
Copy link
Contributor

@Harsh-br0 Harsh-br0 commented Oct 7, 2023

πŸ“š Description

fixes #154

πŸ”– Changes

  • added a fonts.css to link cached font files
  • all cached fonts lies inside docs/public/fonts directory

βœ… To-do list

  • Make sure that all the pipeline passes
  • Make sure you have updated the documentation to reflect the changes or new features
  • Make sure to update the CHANGELOG.md to reflect the new feature or fix
@antonio-gg-dev antonio-gg-dev added documentation Improvements or additions to documentation hacktoberfest labels Oct 7, 2023
Copy link
Member

@antonio-gg-dev antonio-gg-dev left a comment

Choose a reason for hiding this comment

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

Hello, thank you very much for helping us with the project, would you mind making some changes to simplify it?

Since the documentation is only in English, we do not need the versions with the unicode-ranges, the standard version of each font will be sufficient, you can download them from Google Fonts.

Make sure you download the correct weights for each font, we need 400 for Cousine, 600 for Rubik, and for Heebo we need both 400 and 600.

Additionally, it would be great if the filenames of the font files have a name that can be understood.

Thank you very much again for helping us.

@antonio-gg-dev
Copy link
Member

I also noticed that you've updated the yarn.lock, please @Harsh-br0 restore it to the previous version as we have it there precisely to ensure we all have the same versions of the packages and to ensure that the documentation works stably.

@Harsh-br0
Copy link
Contributor Author

Alright, my apologies for responding lately
so i have covered all the requested changes now

@Harsh-br0
Copy link
Contributor Author

Seems like my formatter messed up line endings

@Chemaclass
Copy link
Member

Can you please check the To-do list from the PR itself?
At least the third box: "updating the CHANGELOG" is something you can also add to this PR :)

Screenshot 2023-10-08 at 18 09 34

@Harsh-br0
Copy link
Contributor Author

so, i have checked first 2 checkboxes which makes sense to this PR
But , should i commit the commit messages to CHANGELOG.md then the 3rd checkbox would be valid to this PR ?

@Chemaclass
Copy link
Member

@Harsh-br0, what I would suggest is to add only one line like "Serve fonts from origin instead of cdn" to the Unreleased list from the CHANGELOG. That should be enough

@antonio-gg-dev
Copy link
Member

I believe that since it's not a significant improvement for our users, there's no need to add anything to the changelog, but it would be a shame if your name didn't appear in the next release.

Perhaps you could add something like "optimized docs fonts".

@Harsh-br0
Copy link
Contributor Author

Alright, Now it seems fine

Copy link
Member

@antonio-gg-dev antonio-gg-dev left a comment

Choose a reason for hiding this comment

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

Superb! Thanks you!

@antonio-gg-dev antonio-gg-dev merged commit c1e2004 into TypedDevs:main Oct 8, 2023
@Harsh-br0
Copy link
Contributor Author

Nice, Thanks for such amazing library.
It was good to work with you

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

Labels

documentation Improvements or additions to documentation hacktoberfest

3 participants