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

Update about.html #1635

Closed
wants to merge 1 commit into from
Closed

Update about.html #1635

wants to merge 1 commit into from

Conversation

RishiAnand108
Copy link

Title and Issue number

Title :

Issue No. :

Code Stack :

Close #<issue_no>

Description

Video/Screenshots (mandatory)

Type of PR

  • [ yes] Bug fix
  • [ yes] Feature enhancement
  • [ no] Documentation update
  • Other (specify): _______________

Checklist:

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

Additional context:

mprovements Explained
Responsive Design: Added more flexible CSS classes to improve mobile usability. The hamburger icon toggles the navigation menu on smaller screens.

Clear Call-to-Action (CTA): Added a clear CTA button at the bottom, Start Your Journey, which links directly to the login page for easier onboarding.

Enhanced Navigation Bar: Restructured the navigation bar, emphasizing links, login, and map view buttons. This improves readability and makes it easier to navigate.

Overlay on Video Background: Added an overlay over the video background to improve the readability of text above it.

Feature Descriptions: Added short descriptions for each feature to help users understand the benefits of each service more quickly.

Interactivity with JavaScript: Added a simple JavaScript toggle for the navigation links on mobile.

Visual Hierarchy: Emphasized the main heading (About Us) and subheading, making them stand out. Added spacing and styling to help each section flow better visually.

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

  • I am contributing under GSSOC'24 Extended

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link

netlify bot commented Nov 10, 2024

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
🔨 Latest commit 459ece3
🔍 Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/6730362d09335f00085487d9
😎 Deploy Preview https://deploy-preview-1635--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.

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.

@RishiAnand108 this pr is totally unplanned & there is no issue no. mentioned in the PR & no description is present there. So sorry i can't merge & review this pr. Please try to follow the rules mentioned in CONTRBUTING.md file before contributing

@apu52 apu52 closed this Nov 10, 2024
@apu52 apu52 added wontfix This will not be worked on and removed title needs formatting labels Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
title needs formatting wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants