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

Pro help design #1451

Merged
merged 21 commits into from
Mar 5, 2025
Merged

Pro help design #1451

merged 21 commits into from
Mar 5, 2025

Conversation

idanidan29
Copy link
Contributor

@idanidan29 idanidan29 commented Mar 3, 2025

What kind of change does this PR introduce?

This PR improves the card design on the Pro-Help page to ensure a visually appealing and user-friendly experience across dark mode, light mode, desktop, and mobile. The updated design enhances readability and ensures consistency across different viewing environments.

Closes #1387, Closes #1289

Screenshots/videos:

image

image

and for mobile:

image

image

Summary

This PR improves the card design on the Pro-Help page, enhancing the overall appearance and ensuring that it works well across all conditions, including dark mode, light mode, desktop, and mobile. The update aims to make the cards more visually attractive and user-friendly.

Big thanks to @Thund3rHawk for providing valuable feedback and being a part of the design suggestions.
This design has been discussed in Slack and has been approved by @jdesrosiers

Does this PR introduce a breaking change?

no

@idanidan29 idanidan29 requested a review from a team as a code owner March 3, 2025 10:15
Copy link

github-actions bot commented Mar 3, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview 210cedd

Copy link

codecov bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (6d33bec) to head (210cedd).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1451   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          396       396           
  Branches       106       106           
=========================================
  Hits           396       396           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

I find the full-width justified alignment distracting. Can we change this back to normal left justified?

image

Other than that, it looks good to me.

@Thund3rHawk
Copy link
Contributor

Sure, I suggested because I thought that it would look good but the lots of unnecessary spaces in between words is looking wired.

@idanidan29
Copy link
Contributor Author

@jdesrosiers I changed it back

Copy link
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

That looks better. Thanks.

I noticed another thing. Sorry I missed it the last review. It looks like the "Reach Out" button is rendering in a slightly different place for each bio. I think it should have a fixed size pinned to the left.

image

image

@idanidan29
Copy link
Contributor Author

@jdesrosiers Thanks for the quick reply! It’s fixed now.

Copy link
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks!

@jdesrosiers
Copy link
Member

Hoping to see a review from @json-schema-org/web-team before merging.

Copy link
Collaborator

@benjagm benjagm left a comment

Choose a reason for hiding this comment

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

Great job!!

@benjagm benjagm merged commit 13a4f77 into json-schema-org:main Mar 5, 2025
9 checks passed
Copy link

github-actions bot commented Mar 5, 2025

Congratulations, @idanidan29 for your first pull request merge in this repository! 🎉🎉. Thanks for your contribution to JSON Schema!

@idanidan29 idanidan29 deleted the pro-help-Design branch March 5, 2025 22:06
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.

✨ Enhancement: Revamp the UI for pro-help page 🐛 Bug: More dark mode problems - pro-help
4 participants