Skip to content

Refactor CSS: remove duplicated responsive/modal blocks in readmeforge.css (no UI change)#102

Merged
Mohit-368 merged 2 commits into
Mohit-368:mainfrom
Mayur-Shashidhar:Refactor/CSS
May 15, 2026
Merged

Refactor CSS: remove duplicated responsive/modal blocks in readmeforge.css (no UI change)#102
Mohit-368 merged 2 commits into
Mohit-368:mainfrom
Mayur-Shashidhar:Refactor/CSS

Conversation

@Mayur-Shashidhar
Copy link
Copy Markdown
Contributor

@Mayur-Shashidhar Mayur-Shashidhar commented May 14, 2026

Description

readmeforge.css currently contains duplicated responsive and modal-related style blocks, which increases stylesheet size and makes future maintenance harder.

Changes Made

  • Removed duplicated responsive media query blocks.
  • Removed repeated modal and preview-related styles.
  • Kept a single canonical version of each style section.
  • Preserved existing desktop, tablet, and mobile behavior.
  • Preserved dark/light mode styling.
  • Preserved modal, preview, and print flow functionality.
  • No redesigns, visual changes, or feature additions.

Why This Matters

  • Improves stylesheet maintainability.
  • Reduces risk of inconsistent future edits.
  • Makes the CSS cleaner and easier to navigate.
  • Prevents accidental regressions caused by duplicated rules.

Acceptance Criteria

  • Duplicate responsive and modal blocks are removed.
  • UI behavior remains unchanged in both dark and light mode.
  • Modal, preview panel, and print flow continue working as before.
  • No layout breakage across mobile/tablet/desktop breakpoints.

GSSoC 2026

This PR is a part of the program GirlScript Summer of Code 2026.
Issue is #86

@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for makeareadme ready!

Name Link
🔨 Latest commit fdcb8ec
🔍 Latest deploy log https://app.netlify.com/projects/makeareadme/deploys/6a06ffb4bda7f500083f12ba
😎 Deploy Preview https://deploy-preview-102--makeareadme.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@Mayur-Shashidhar
Copy link
Copy Markdown
Contributor Author

@Mohit-368 Please review the pr and add the neccessary labels so that i can gain my points
Thank you

@Mayur-Shashidhar
Copy link
Copy Markdown
Contributor Author

Screenshot 2026-05-15 at 4 46 38 PM Screenshot 2026-05-15 at 4 46 45 PM Screenshot 2026-05-15 at 4 46 52 PM @Mohit-368 Even after complete update to React + Vite even before I put in my PR its working and no chages in UI

@Mohit-368 Mohit-368 merged commit db97eba into Mohit-368:main May 15, 2026
8 checks passed
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.

2 participants