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

Fixed issue of Spacing above the heading #173

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MouliiiSharma
Copy link

@MouliiiSharma MouliiiSharma commented Jun 5, 2024

PR Description 📜

Issue #147 that was to enhance the view of page by fixing the heading " Help us improve!" is done now
Fixed # <#147>

  • I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
  • I have performed a self-review of my own code or work.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generates no new warnings.
  • I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
  • I have added screenshot for review.

image

@MouliiiSharma
Copy link
Author

please check this

@thevirengarg
Copy link
Collaborator

@MouliiiSharma Spacing above the heading will be different for every laptop's screen size ratio. Update it in such a way it should be consistent for every screen size.

@MouliiiSharma
Copy link
Author

okay sir

@MouliiiSharma
Copy link
Author

Now I have changed the heading space in a way so that it will be consistent for every screen size and as content was overflowing I have added a scroll bar too. please check this .

physcitech.mp4

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