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

Enhance Responsiveness and Accessibility with CSS Updates #384

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

Conversation

pcodesdev
Copy link

Description

This pull request focuses on improving the flexibility and accessibility of the portfolio website by updating CSS files to use responsive units. The changes affect all CSS files in the components folder, as well as the main index.css and app.css files.

Changes Made

  1. Updated all CSS files in the components/ directory to use responsive units
  2. Modified index.css to incorporate responsive units
  3. Adjusted app.css to enhance flexibility and accessibility

Benefits

  • Improved responsiveness across different screen sizes and devices
  • Enhanced accessibility for users with different zoom levels or font size preferences
  • Better overall flexibility of the application's layout

Testing

Please review these changes on various devices and screen sizes to ensure proper responsiveness and layout integrity.

Additional Notes

This update is part of my ongoing effort to improve the user experience and adhere to modern web development best practices.

…esponsive units for improved flexibility and accessibility
@pcodesdev
Copy link
Author

The conflicting part on src/components/talkCard/TalkCard.css the rectangle class selector should be edited to:
.rectangle { background: url(..\..\assets\images\talksCardBack.svg); width: 23.75rem; /* Changed from 380px to 23.75rem */ height: 18.75rem; /* Changed from 300px to 18.75rem */ border-radius: 0.25rem; /* Changed from 4px to 0.25rem */ box-shadow: 0 1.25rem 3.125rem #d9dbdf; /* Changed from 0px 20px 50px to 0 1.25rem 3.125rem */ -webkit-transition: all 0.3s ease; -o-transition: all 0.3s ease; transition: all 0.3s ease; margin-top: 3.125rem; /* Changed from 50px to 3.125rem */ }

@saiteja13427
Copy link
Collaborator

Seems like you pushed some files other than the css files as well, it shows 1726 file changes?

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