Skip to content

Conversation

sohan9819
Copy link
Contributor

Completed making the idea of full screen navbar

Here is the preview

Fullscreen Overlay Navigation

Tech Stack Used

html5 css3 javascript

Preview

fullscreen_navbar

Completed making the idea of full screen navbar
Comment on lines 9 to 20
body {
font-family: 'Playfair Display', serif;
background: #fdfdea;
transition: all 2s ease;
}
h1 {
font-size: 4rem;
font-weight: 400;
}
h3 {
font-size: 1.5em;
}
Copy link
Member

Choose a reason for hiding this comment

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

don't add styling directly to HTML tags, use classes instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@thepranaygupta did the changes 👍.

Applied css to classes of tags instead of applying direct to tag names
Copy link
Member

@nandini7637 nandini7637 left a comment

Choose a reason for hiding this comment

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

Just add a class to body tag too else everything looks great

@sohan9819
Copy link
Contributor Author

@nandini7637 done 👍 ! added class to body

@nandini7637 nandini7637 merged commit fe07057 into Dezenix:main Jan 13, 2022
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.

6 participants