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

Changed font style #564

Merged
merged 5 commits into from
Jun 26, 2023
Merged

Changed font style #564

merged 5 commits into from
Jun 26, 2023

Conversation

DivyanshiNarang
Copy link
Contributor

Resolves issue #505

Changed font style to maintain the consistency of the page.

Before:
Screenshot 2023-06-24 235652

Now:
Screenshot 2023-06-24 235156

@gurjeetsinghvirdee
Copy link

gurjeetsinghvirdee commented Jun 25, 2023

I mean really you are assigning font fixing issues @narayan954

Definitely, I will check your repo PRs day after tomorrow.

BcfdMonkeyGIF

@narayan954
Copy link
Owner

I mean really you are assigning font fixing issues @narayan954

Definitely, I will check your repo PRs day after tomorrow.

BcfdMonkeyGIF

that's what I mentioned, I'll no longer allow single line issues.

Copy link
Owner

@narayan954 narayan954 left a comment

Choose a reason for hiding this comment

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

check comment

src/components/Navbar/index.jsx Outdated Show resolved Hide resolved
Removed fontFamily attribute as this property is inherited from root/index.js file.
Copy link
Owner

@narayan954 narayan954 left a comment

Choose a reason for hiding this comment

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

looks good to me!

@narayan954 narayan954 merged commit 3f3c128 into narayan954:master Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants