Feature adding logo in skill section - #1160
Open
Ash-Misty wants to merge 3 commits into
Open
Conversation
|
@Ash-Misty is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
Owner
|
@Ash-Misty the feature looks good, but you will need to resolve the conflicts first before i merge |
Author
|
solved |
Owner
|
@Ash-Misty please check again |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary [required]
Replaced text-based skill names in the Skills section with technology logos to improve visual clarity, modern UI consistency, and overall user experience.
Related Issue [required]
Closes #1126
Type of Change [required]
data/projects.jsonWhat Was Changed [required]
How to Test This PR [required]
Install dependencies (if applicable):
npm install
OR
pip install -r requirements.txt
Run the project:
npm run dev
OR
python app.py
Open in browser:
http://127.0.0.1:3000
Navigate to Skills section and verify:
Skill logos are displayed instead of text
Layout is properly aligned
No overflow or spacing issues
Works properly on mobile (375px) and desktop (1280px)
Test Results [required]
UI tested manually on:
Results:
##Screenshots

Self-Review Checklist [required]
feat/,fix/,docs/,style/,test/)flake8 .or equivalent) with no errorsprint,console.log)Notes for Reviewer
This is a pure UI/UX improvement with no logic changes.
Skills section now uses icon-based representation for better visual clarity.
Please verify responsiveness and alignment across all screen sizes.