profileme-dev icon indicating copy to clipboard operation
profileme-dev copied to clipboard

Light Screen Icons

Open FREDVUNI opened this issue 1 year ago • 11 comments

I have a question and an inquiry. When I am in light mode, some icons are missing, but everything appears fine in dark mode. Did I set something up incorrectly, or is this a feature that doesn’t exist yet? Screenshot 2024-08-09 155354

FREDVUNI avatar Aug 09 '24 12:08 FREDVUNI

As you can seen the image, a couple of icons are missing

FREDVUNI avatar Aug 09 '24 12:08 FREDVUNI

Which icons are missing? I've just tried this myself at 3fc3595593bc992e6febba6580d6c9571f5e625f using npm run dev and didn't encounter any issues with icons.

LachyTech avatar Aug 14 '24 06:08 LachyTech

In dark mode, they all appear image

Icons missing: Next JS, Babel, Django, XD and Git hub

FREDVUNI avatar Aug 14 '24 19:08 FREDVUNI

Are you hosting the project yourself in a dev environment, or using the production version at profileme.dev?

There seems to be a link between how the project is deployed and how some image links are generated, at least for the social ones. I'm still getting familiar with how it works.

LachyTech avatar Aug 19 '24 01:08 LachyTech

I'm using profileme.dev

FREDVUNI avatar Aug 19 '24 07:08 FREDVUNI

I see the issue you're experiencing now, it isn't that the social icons are missing (that's another issue), but instead it's the fact that the markdown content only generates for one colour-scheme for skill icons based on whether you choose light or dark mode in the preview. If you choose dark mode in the profileme.dev preview, then apply to github and set your preference to light mode, some icons will be indistinguishable from the background.

It does the correct behaviour for the socials where it includes the <source media="(prefers-color-scheme: dark)" tags, so we'd need to do something similar for skill icons where the path/darkPath attributes are set.

Light mode on github: image

Dark mode on github: image

LachyTech avatar Aug 20 '24 04:08 LachyTech

Oh, thanks. will definitely be on the look out for that!

FREDVUNI avatar Aug 20 '24 14:08 FREDVUNI

There's still a fix here worth doing to address the issue. :)

LachyTech avatar Aug 21 '24 03:08 LachyTech

Could you maybe share, @LachyTech ??

FREDVUNI avatar Aug 21 '24 16:08 FREDVUNI

I've just pushed a PR to address the issue. Not sure how long it will take to get reviewed and accepted, but you can deploy manually to get the output now if you need to.

LachyTech avatar Aug 21 '24 21:08 LachyTech

Cool, thanks @LachyTech

FREDVUNI avatar Aug 22 '24 06:08 FREDVUNI