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

feat: Remove Overflow Hidden and Enhance Navbar Styling #1009

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SaiTejaswaniBikkasani
Copy link
Contributor

Fixed issue: #1005

Title and Issue number

Title : #1005

Issue No. : #1005

Code Stack : HTML, CSS

Close #1005

Description

The current web page design has overflow: hidden, preventing access to the entire page content. This commit modifies the CSS to allow users to scroll and access the entire page content.
Improved the navbar by making the following changes:

  • Ensured the cursor changes to a pointer when hovering over the navigation links and also added the hover effects to the links in the navbar for better user experience.
  • Added proper spacing and padding to the navigation links to prevent them from connecting to the margin.

Video/Screenshots (mandatory)

Screen.Recording.2024-07-07.120811.mp4

Type of PR

  • Feature enhancement

Checklist:

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have gone through the contributing.md file before contributing

Additional context:

##Are you contributing under any Open-source programme?

  • I am contributing under GSSOC'24

The current web page design has overflow: hidden, preventing access to the entire page content. This commit modifies the CSS to allow users to scroll and access the entire page content and improved the navbar
Copy link

netlify bot commented Jul 7, 2024

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
🔨 Latest commit 01ef0fd
🔍 Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/668a38b982deb600086f84a4
😎 Deploy Preview https://deploy-preview-1009--taupe-cendol-f7e2bb.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@apu52 apu52 changed the title Remove Overflow Hidden and Enhance Navbar Styling feat: Remove Overflow Hidden and Enhance Navbar Styling Jul 13, 2024
Copy link
Owner

@apu52 apu52 left a comment

Choose a reason for hiding this comment

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

@SaiTejaswaniBikkasani I can't see any of your changes in the PR itself as it shows me this
image

I think maybe the newplantrip.html page is not connected with the main page so that none of your changes is shown up to me .please check it & fix it

@apu52 apu52 added the help wanted Extra attention is needed label Jul 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Enable Page Scrolling and Improve Navbar Design in newplantrip page
2 participants