Skip to content

Comments

Replaced Custom Icon component with react-icons#456

Merged
its-me-abhishek merged 2 commits intoCCExtractor:mainfrom
Neeraj-gagat:feat/alternative-icon
Feb 5, 2026
Merged

Replaced Custom Icon component with react-icons#456
its-me-abhishek merged 2 commits intoCCExtractor:mainfrom
Neeraj-gagat:feat/alternative-icon

Conversation

@Neeraj-gagat
Copy link
Contributor

@Neeraj-gagat Neeraj-gagat commented Feb 5, 2026

Description

This PR changes the customs icons used in how it works section with react-icons as sugested in issue descriptions #451

Images after changes

image

Additional Notes

hey @its-me-abhishek
if you want we can discuss if you want to use any other icons or any other icons library open to suggestions
Thanks :)

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

More information on how to conduct a self review:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

Copy link
Contributor Author

@Neeraj-gagat Neeraj-gagat left a comment

Choose a reason for hiding this comment

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

done with the self review every test passes unused icons removed new icons added nothing breaks goood to go

@its-me-abhishek
Copy link
Collaborator

@Neeraj-gagat I think we can delete Icons.tsx entirely, since Logo Icon is unused

@Neeraj-gagat
Copy link
Contributor Author

okk sure will update the pr in a while

@Neeraj-gagat
Copy link
Contributor Author

@Neeraj-gagat I think we can delete Icons.tsx entirely, since Logo Icon is unused

@Neeraj-gagat I think we can delete Icons.tsx entirely, since Logo Icon is unused

done

Copy link
Collaborator

@its-me-abhishek its-me-abhishek left a comment

Choose a reason for hiding this comment

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

LGTM

@its-me-abhishek its-me-abhishek merged commit 425c269 into CCExtractor:main Feb 5, 2026
5 checks passed
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.

HowItWorks - Dashboard frontend revamp

2 participants