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

Updating Sliders and Table Ehancement #460

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

Conversation

arjunxplorer
Copy link
Contributor

-Implemented a smoother scrolling behavior and adjusted the styling to ensure better visibility and usability.
-I've revamped the table layout, refining the design and optimizing it for readability and accessibility.

@arjunxplorer arjunxplorer self-assigned this Apr 11, 2024
Copy link

netlify bot commented Apr 11, 2024

Deploy Preview for spacelab-main ready!

Name Link
🔨 Latest commit f7ade7f
🔍 Latest deploy log https://app.netlify.com/sites/spacelab-main/deploys/661ee7e0193ef30008bddbe3
😎 Deploy Preview https://deploy-preview-460--spacelab-main.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.

@aanand93 aanand93 self-requested a review April 11, 2024 19:33
Copy link
Collaborator

@aanand93 aanand93 left a comment

Choose a reason for hiding this comment

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

Arjun, Great PR. You code is clean and efficient and the slider is smooth and works appropriately.

Can you make the slider thinning and more see through. It is very harsh comapred to the rest of the layout. Also the color scheme should match the purple and blues we use throughout the site. If you can:

  1. fix the colors to match the theme, check out some of the color codes used in the source code.
  2. thin out the slider and lower the opacity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: PR REVIEW: REQUESTED CHANGES
Development

Successfully merging this pull request may close these issues.

2 participants