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 main.css #6396

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

Conversation

Knowledgelocater
Copy link

1 --> Viewport-based (vw) Units: Replaced most fixed px values with vw to ensure responsiveness. 2 --> Media Queries:
Adjustments for smaller screens (mobile).
Enhancements for larger screens (desktop).
3--> Improved Readability:
Adjusted line-height and spacing.
Added responsive font sizes for better UX on different devices. 4--> Scalable Layout: Made carousel, navbar, and other components more flexible.

1 --> Viewport-based (vw) Units: Replaced most fixed px values with vw to ensure responsiveness.
2 --> Media Queries:
Adjustments for smaller screens (mobile).
Enhancements for larger screens (desktop).
3--> Improved Readability:
Adjusted line-height and spacing.
Added responsive font sizes for better UX on different devices.
4--> Scalable Layout: Made carousel, navbar, and other components more flexible.
@Knowledgelocater
Copy link
Author

This main.css file has been updated to enhance the responsiveness of the website. The px units have been replaced with vw units to allow for greater adaptability across different screen sizes. Media queries have been added to ensure the website adjusts seamlessly to various devices. Additionally, the code has been refactored to improve readability and maintainability, aligning with professional standards.

This contribution aims to support the project's scalability and accessibility while adhering to best practices in responsive web design.

@khancyr
Copy link
Contributor

khancyr commented Nov 18, 2024

hello,

Thank for the inputs. But it looks like some AI stuff without checking :
Master
master1
PR
PR1

Master
master2
PR
PR2

Master is left
test

comp1
comp2

So I don't think I will merge and it makes things worse IMHO

@Knowledgelocater
Copy link
Author

I have used chatgpt but rechecked on my own it was working

@Knowledgelocater
Copy link
Author

@khancyr I used formula ,vw = (max-width / viewport width) * 100 for the conversion on px to vw
and media queries for more screen type

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