Skip to content

Add amro#78

Merged
amro-youssef merged 5 commits intomainfrom
amro
Feb 5, 2026
Merged

Add amro#78
amro-youssef merged 5 commits intomainfrom
amro

Conversation

@amro-youssef
Copy link
Contributor

Added amro to the portfolio page. May need to fix some merge conflicts

@amro-youssef amro-youssef removed the request for review from orangespaceman February 5, 2026 14:41
@amro-youssef
Copy link
Contributor Author

dont merge yet, needs to be fixed

@amro-youssef
Copy link
Contributor Author

should be fixed now

@amro-youssef amro-youssef added the enhancement New feature or request label Feb 5, 2026
Copy link
Member

@orangespaceman orangespaceman left a comment

Choose a reason for hiding this comment

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

All good - there is one small fix needed, but after that it will be good to merge! 👏

.card--amro {
/* background: linear-gradient(340deg, #2a7b9b 0%, #00ad48 100%); */

.card--finn svg:hover {
Copy link
Member

Choose a reason for hiding this comment

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

It looks like this one style might have been removed:

.card--finn svg:hover {
    filter: drop-shadow(5px 5px 5px #005c30);
    transform: scale(1.5);
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops, didn't mean to sabotage Finn 🤣

@amro-youssef amro-youssef merged commit a6ce1c9 into main Feb 5, 2026
1 check passed
@amro-youssef amro-youssef deleted the amro branch February 5, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants