Skip to content

Adding LinkedIn to the council profile#106

Merged
notkaramel merged 3 commits intomainfrom
antoine/council-linkedin
Feb 15, 2026
Merged

Adding LinkedIn to the council profile#106
notkaramel merged 3 commits intomainfrom
antoine/council-linkedin

Conversation

@notkaramel
Copy link
Member

Also added LinkedIn + URL Validation on the CMS

@notkaramel notkaramel self-assigned this Feb 11, 2026
@vercel
Copy link

vercel bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ecsess Ready Ready Preview, Comment Feb 11, 2026 7:40pm

{/if}
{#if linkedin}
<div class="flex items-center justify-center gap-2">
<Linkedin class="stroke-ecsess-400 size-4 stroke-2" aria-hidden="true" />
Copy link
Contributor

Choose a reason for hiding this comment

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

add fill="white" cuz the filling is empty for the icon

Copy link
Member Author

Choose a reason for hiding this comment

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

Lucide icon filling is empty by default :(

Copy link
Contributor

Choose a reason for hiding this comment

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

yea but u can add filling in. I did that for the footer linkedIn logo. It's just me being picky bleh

Copy link
Contributor

@MinhVo2005 MinhVo2005 left a comment

Choose a reason for hiding this comment

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

Can you make it so that when hover on the x button to exit, it should change cursor

@notkaramel
Copy link
Member Author

Can you make it so that when hover on the x button to exit, it should change cursor

Done!

@notkaramel notkaramel merged commit 28ba5fd into main Feb 15, 2026
4 checks passed
@notkaramel notkaramel deleted the antoine/council-linkedin branch February 15, 2026 18:49
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.

2 participants