Skip to content

Conversation

@adity1raut
Copy link
Contributor

📌 Fixes

Fixes #273


📝 Summary of Changes

  • Short description of what was changed
  • Include links to related issues/discussions if any

Changes Made

Screenshot From 2025-11-13 09-22-23
  • Updated ...
  • Refactored ...
  • Fixed ...
  • Added tests for ...

Checklist

Please ensure the following before submitting your PR:

  • I have reviewed the project's contribution guidelines.
  • I have written unit tests for the changes (if applicable).
  • I have updated the documentation (if applicable).
  • I have tested the changes locally and ensured they work as expected.
  • My code follows the project's coding standards.

Screenshots or Logs (if applicable)


👀 Reviewer Notes

Add any special notes for the reviewer here

@adity1raut
Copy link
Contributor Author

PTAL @btwshivam

@oksaumya
Copy link
Member

give space in between

@SakshamDutt
Copy link
Collaborator

@adity1raut Design is looking good but it's not responsive for smaller widths. Fix this. You can take inspiration from the present footer.

Signed-off-by: adity1raut <[email protected]>
@adity1raut
Copy link
Contributor Author

Screenshot From 2025-11-15 13-37-27

PTAL @oksaumya , @naman9271

@oksaumya
Copy link
Member

oksaumya commented Nov 15, 2025

Looks good. Make a few more changes add “Enter your email address” in the email input placeholder and
Screenshot 2025-11-15 at 5 06 03 PM
improve this We still have plenty of space available, so we can wrap the sections more effectively.

Signed-off-by: adity1raut <[email protected]>
@github-actions github-actions bot added the json label Nov 15, 2025
@adity1raut
Copy link
Contributor Author

Screenshot From 2025-11-15 18-20-43 Screenshot From 2025-11-15 18-21-08

@adity1raut
Copy link
Contributor Author

PTAL @oksaumya , @naman9271

@btwshivam
Copy link
Collaborator

@adity1raut Keep place holder is Email only. remove We respect your privacy. No spam. these text. keep stay updated in same line. And for docs, getting started and resource manege the spacing and layout. overall i want it to be professional

@btwshivam
Copy link
Collaborator

I liked the image in PR des. work on that

@adity1raut
Copy link
Contributor Author

Screenshot From 2025-11-16 08-40-44 Screenshot From 2025-11-16 08-40-57

PTAL @btwshivam

@btwshivam
Copy link
Collaborator

Very small change start it from docs alignment and end it resources alignment on pc screen

@adity1raut
Copy link
Contributor Author

Very small change start it from docs alignment and end it resources alignment on pc screen

Any more Changes Much Want

@adity1raut
Copy link
Contributor Author

Screenshot From 2025-11-16 09-47-35

PTAL @btwshivam

Signed-off-by: adity1raut <[email protected]>
@github-actions github-actions bot removed the json label Nov 16, 2025
@oksaumya
Copy link
Member

oksaumya commented Nov 19, 2025

Resolve the conflicts and make the spacing same between “Stay Updated” and the email box, and between “Subscribe” and the email box.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature: Remove Stay Updated and place it below downside the sections and Increse the length of input box

4 participants