Skip to content

Conversation

@ShreyJaiswal1
Copy link

πŸ”„ Pull Request

πŸ“‹ Type of Change (check all applicable)

  • πŸ› Bug Fix - Fixes an issue without breaking existing functionality
  • ✨ Feature - Adds new functionality
  • ⚑ Performance - Improves performance without changing functionality
  • ♻️ Refactor - Code changes that neither fix bugs nor add features
  • πŸ“š Documentation - Updates to documentation, comments, or README
  • 🎨 Style - Code style changes (formatting, missing semi-colons, etc.)
  • πŸ§ͺ Test - Adding or updating tests
  • πŸ—οΈ Build - Changes to build system or dependencies
  • πŸ‘· CI/CD - Changes to CI/CD workflows
  • πŸ”’ Security - Security improvements or vulnerability fixes
  • β™Ώ Accessibility - Improves accessibility compliance
  • πŸ“± Mobile - Mobile-specific improvements
  • 🌐 i18n - Internationalization changes

πŸ“– Description

What changed?

Added more platfroms in skills section like
n8n, make, playwrite, maya 3d.
and fixed next.js category from 'static site generator' to 'frontend'

Why was this change made?

to choose more skills

How does this change help users?

by adding more choices

πŸ”— Related Issues

πŸ§ͺ Testing & Quality Assurance

Testing Done (check all applicable)

  • βœ… Manual testing - Tested functionality manually
  • πŸ§ͺ Unit tests - Added/updated unit tests
  • πŸ”„ Integration tests - Tested with other components
  • πŸ“± Mobile testing - Tested on mobile devices
  • β™Ώ Accessibility testing - Tested with screen readers/keyboard nav
  • 🌐 Cross-browser testing - Tested in multiple browsers
  • 🎨 Visual testing - Checked UI/UX in light/dark themes

Test Instructions

  1. Run the repo

Expected Behavior

πŸ“Έ Screenshots/Recordings

Before

image

After

image image image

πŸ“‹ Checklist

Code Quality

  • πŸ” TypeScript - No TypeScript errors (npm run type-check)
  • 🧹 Linting - No ESLint errors (npm run lint)
  • 🎨 Formatting - Code is properly formatted (npm run format)
  • πŸ—οΈ Build - Production build succeeds (npm run build)
  • ⚑ Performance - No performance regressions introduced

Accessibility

  • β™Ώ WCAG Compliance - Follows WCAG 2.1 AA guidelines
  • ⌨️ Keyboard Navigation - All interactive elements are keyboard accessible
  • πŸ” Screen Reader - Proper ARIA labels and semantic HTML
  • 🎨 Color Contrast - Meets contrast requirements
  • 🎯 Focus Management - Visible focus indicators

Mobile & Responsive

  • πŸ“± Mobile Responsive - Works on mobile devices (320px+)
  • πŸ–₯️ Desktop - Works on desktop (1024px+)
  • πŸ“ Tablet - Works on tablet sizes (768px+)
  • πŸ”„ Orientation - Works in portrait and landscape

Browser Compatibility

  • 🌐 Chrome - Latest version
  • 🦊 Firefox - Latest version
  • 🧭 Safari - Latest version
  • πŸ“± Mobile Safari - iOS Safari
  • πŸ“± Chrome Mobile - Android Chrome

Documentation

  • πŸ“š Code Comments - Added helpful comments for complex logic
  • πŸ“– Documentation - Updated relevant documentation
  • πŸ“ README - Updated README if needed
  • πŸ”„ Changelog - Will be auto-generated from conventional commits

Security & Privacy

  • πŸ”’ No Secrets - No API keys, passwords, or sensitive data exposed
  • πŸ›‘οΈ Input Validation - Proper validation for user inputs
  • πŸ” XSS Prevention - Protected against XSS attacks
  • πŸͺ Privacy Compliant - Follows GDPR/privacy requirements

πŸš€ Deployment Notes

  • No breaking changes - Backward compatible
  • Database changes - N/A (static site)
  • Environment variables - No new env vars needed
  • Third-party dependencies - No new external dependencies

πŸ“ Additional Notes

πŸ‘€ Reviewers


By submitting this PR, I confirm that:

Copy link
Owner

Choose a reason for hiding this comment

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

this icon will not work in dark mode

@ShreyJaiswal1
Copy link
Author

Yeah ill update the icons soon. by the way. these icons look colorless like ifttt one. they appear as colored one because we fetch them via skill simpleicons website. i have tested these icons in dark mode only. as attached screenshot

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.

Add more skills choice

2 participants