Refine Community Engagement Section Alignment and Spacing#1786
Refine Community Engagement Section Alignment and Spacing#1786srushti-panara wants to merge 11 commits into
Conversation
Refresh the community contribution cards with clearer structure, richer styling, and responsive interactive states to create a more polished UX. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
:wq# the commit.
|
@srushti-panara is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
|
✅ Synchronized metadata from Issue #1771:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| @@ -1,4 +1,4 @@ | |||
| import React, { useState, useEffect } from "react"; | |||
| import React, { useEffect, useState } from "react"; | |||
| import Layout from "@theme/Layout"; | |||
There was a problem hiding this comment.
Hi not trying to discourage, the way you contributed is amazing, especially feature upgrade push on different commits.
this shows you have spent some time on this pr.
but the expected behaviour didnt match.
This is completly revamp of the page and reduced readability, + vibe coded feel to it.
So in future any UI change, lets dicusss more changes in the same issue along with development.
because once it go production many people will use it so as a product perspective its not good.
https://recode-website-h8imx1wn7-recode-9c2fd7df.vercel.app/community
so here is what you can do i will be rejecting this PR. but you can raise new pr with change reademe.md by doing a
and i will merge and give same level 3 and merge it for your effort
cc: @Adez017
dont get discouraged, this is part of opensource journey
There was a problem hiding this comment.
Thank you for the detailed feedback and encouragement. I really appreciate you taking the time to review my work and explain the concerns from a product perspective.
This was a valuable learning experience for me, especially understanding the difference between refining an existing UI and unintentionally redesigning it. I'll definitely discuss larger UI changes more thoroughly before implementation in the future.
Regarding the README contribution you mentioned, could you please guide me on which section you would like me to improve or update? I'd be happy to work on it and raise a new PR.
Also, for future contributions, if I limit the scope to minor UI refinements such as alignment, spacing, responsiveness, and subtle interaction improvements while preserving the existing design language, would it be appropriate to create a new PR for similar UI-related issues?
Thank you again for the guidance and support.
There was a problem hiding this comment.
@srushti-panara you can change a word in readme or adding a space works
Description
This PR refines the Community Engagement section by improving alignment, spacing, hover interactions, and overall visual consistency. The changes enhance readability, improve component positioning, and provide a more polished user experience.
Fixes #1771
Type of Change
Changes Made
Dependencies
Checklist
npm run buildand attached screenshot(s) in this PR.Before Screenshot
After Screenshot
Before Screenshot
After Screenshot