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

Initial Steps in Migrating CSS to Tailwind for Footer Component #149

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

Conversation

prajwaln07
Copy link
Contributor

πŸ› οΈ Fixes Issue

Closes #133

πŸ‘¨β€πŸ’» Description

  • This pull request addresses the ongoing migration of CSS styles to Tailwind CSS for the project.
  • I have initiated the process by converting specific components, including the Footer component, to utilize Tailwind CSS classes instead of traditional CSS.
  • This transition aims to enhance the maintainability and scalability of the project's styles, allowing for easier modifications and a more responsive design.

πŸ“„ Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (adds or updates related documentation)

πŸ“· Screenshots/GIFs (if any)

Include screenshots or GIFs to demonstrate your changes

βœ… Checklist

  • I am a participant of GSSoC-ext.
  • I have followed the contribution guidelines of this project.
  • I have viewed deployment of my code.
  • My changes generate no new warnings.
  • I have added documentation to explain my changes.

🀝 GSSoC Participation

  • This PR is submitted under the GSSoC program.
  • I have taken prior approval for this feature/fix.

πŸ› οΈ Future Work

  • Given the extensive nature of this migration, I plan to continue migrating remaining styles to Tailwind CSS in subsequent pull requests. This phased approach will help minimize conflicts and maintain project stability.

Copy link

netlify bot commented Oct 12, 2024

πŸ‘· Deploy request for finnews pending review.

Visit the deploys page to approve it

Name Link
πŸ”¨ Latest commit 71970ca

Copy link

vercel bot commented Oct 12, 2024

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
fin-news βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Oct 12, 2024 11:12am
fin-news-backend βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Oct 12, 2024 11:12am

@prajwaln07
Copy link
Contributor Author

@divyansh-2005

πŸ“ Note on Conflicts

As I continue to migrate the CSS styles to Tailwind CSS, I anticipate that there may be potential conflicts with ongoing developments in the repository. To manage this, I plan to submit the migration in multiple stages, allowing for easier integration and resolution of conflicts.

I appreciate your understanding and am open to any suggestions on how to approach this transition smoothly.

@prajwaln07 prajwaln07 changed the title Tailwind Initial Steps in Migrating CSS to Tailwind for Footer Component Oct 12, 2024
@DiwareNamrata23
Copy link
Collaborator

@divyansh-2005 look on to it

@prajwaln07
Copy link
Contributor Author

prajwaln07 commented Oct 14, 2024

@DiwareNamrata23
hi @divyansh-2005 ,

Since the CSS to Tailwind CSS migration is quite extensive, I suggest proceeding in stages. This way, we can address potential conflicts and ensure a smoother transition. I’ll submit PRs for each component or section, review them incrementally, and resolve any issues as we go.

Let me know your thoughts!

Thanks! 😊

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.

FEATURE : Migrate CSS Styles to Tailwind CSS
2 participants