nextjs-notion-starter-kit
nextjs-notion-starter-kit copied to clipboard
Add twitterX, Gitlab, Instagram, Reddit and Bugtracker social links
This is part of chain of PRs which I would like to merge as small changes so that they are easier to review. Please start from #642 and then proceed to #644.
After that we should be ready to merge these new social icons.
This replaces #591.
Demo video
https://github.com/transitive-bullshit/nextjs-notion-starter-kit/assets/5691777/499e4473-3ea8-49e0-9c4d-9c607c414477
@onnimonni is attempting to deploy a commit to the Saasify Team on Vercel.
A member of the Team first needs to authorize it.
Removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@react-icons/[email protected]
Pull Request Review: Add twitterX, Gitlab, Instagram, Reddit and Bugtracker social links
Thank you for this pull request! I've reviewed the changes to add additional social media links to the project.
Overall Impression
This PR adds several new social media link options including TwitterX, GitLab, Instagram, Reddit, and Bugtracker. The implementation looks well thought out and appears to build on previous refactoring work that improved the social icons system.
Positive Aspects
- Good refactoring approach by consolidating social icon definitions and colors in one place
- Elimination of
dangerouslySetInnerHTMLin favor of safer CSS approaches - Nice implementation of Instagram's gradient background using a custom background setting
- Well-organized commits with descriptive messages
- Includes a demo video, which is helpful for visual confirmation
Things to Consider
-
Package Management: Replacing
@react-icons/all-fileswithreact-iconsseems like a good move for more up-to-date icons, but did you check for any bundle size implications? -
Backward Compatibility: Are there any breaking changes for users who might have configured the previous social links? It's worth documenting any migration steps if needed.
-
Documentation: Consider updating the documentation to reflect the new social media options and how to configure them.
-
Testing: Have all the new social links been tested across different browsers and screen sizes?
-
Internationalization: Are there any text elements in these new social icons that might need translation?
Code Questions
- For the Instagram gradient background, have you tested how it renders on different display types (e.g., dark mode)?
- Is there a reason for the specific ordering of the social icons? Consider if there should be a way for users to customize the order.
Conclusion
This looks like a valuable addition to the project that will give users more flexibility in configuring their social links. The code changes appear well-structured, but please consider addressing the questions and considerations above.
I'd recommend merging this after ensuring the documentation is updated and any necessary testing is completed.