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

Change design system Hebrew font to rubik #525

Merged
merged 3 commits into from
Feb 13, 2022

Conversation

hadasfa
Copy link
Contributor

@hadasfa hadasfa commented Feb 9, 2022

Basic

  • Used plop (npm run plop) to create a new component.
  • PR has description.
  • New component is functional and uses Hooks.
  • Component defines PropTypes.

Style

  • Styles are added to NewComponent.modules.scss file inside of the NewComponent folder.
  • Component uses CSS Modules.
  • Design is compatible with Monday Design System.

Storybook

  • Stories were added to /src/NewComponent/__stories__/NewComponent.stories.js file.
  • Stories include all flows of using the component.

Tests

@@ -1,5 +1,8 @@
<link href="https://fonts.googleapis.com/css?family=Roboto:100,200,300,400,500" rel="stylesheet" />
<link href="http://fonts.cdnfonts.com/css/sofia-pro" rel="stylesheet" />
<link href="https://fonts.googleapis.com/css2?family=Rubik:ital,wght@0,300;0,400;0,500;0,700;1,300;1,400;1,500;1,700&display=swap" rel="stylesheet">
Copy link
Contributor

Choose a reason for hiding this comment

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

Where are we actually using it though? This is just the loading part.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there is css variable which called font-family and other variables which using it (look in the typography.scss file).
the reason the font is updated is because we update monday-ui-style

Copy link
Contributor

Choose a reason for hiding this comment

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

There are 2 typography.scss files in this repo, and I couldn't understand where the font is used exactly.

Can you point me to the specific line where it's used, or try to explain in more detail?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure.
there is 2 typography.scss files: one for the system and our components typography.scss and one for the storybook itself.
you can see in this file mixins we using our font variables https://github.com/mondaycom/monday-ui-react-core/blob/master/src/styles/typography.scss

@hadasfa hadasfa merged commit 9a81f72 into master Feb 13, 2022
@hadasfa hadasfa deleted the fonts/hadasfa/hebrew-update branch February 13, 2022 09:42
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.

2 participants