spartan icon indicating copy to clipboard operation
spartan copied to clipboard

Feature - making the image in The 300 clickable along with the text

Open monacodelisa opened this issue 11 months ago • 6 comments

Which scope/s are relevant/related to the feature request?

Docs

Information

When I look at The 300 each contributor has the Spartan logo image and their username below only the username is clickable

image

the text is a bit tiny and I feel that it will a bit more user friendly if both the image and the text are wrapped up in the link element, so that clicking on either would navigate the user to the contributor's GitHub profile

Describe any alternatives/workarounds you're currently using

No response

I would be willing to submit a PR to fix this issue

  • [X] Yes
  • [ ] No

monacodelisa avatar Mar 24 '24 11:03 monacodelisa

@monacodelisa - just wanted to double check if you planned on making this change yourself? Otherwise this could make a good first issue for someone.

thatsamsonkid avatar Mar 25 '24 22:03 thatsamsonkid

@thatsamsonkid sure, this one as well - maybe keep it open, and if no activity after a while, I can take it then.

monacodelisa avatar Mar 26 '24 03:03 monacodelisa

i'll take the issue if you guys don't mind

mateoetchepare avatar Mar 26 '24 18:03 mateoetchepare

sorry, i had a lot of trouble trying to run the development server bc i was on a windows machine, i justed fixed it by installing a debian VM. i promise in a few days i'll have the PR. thank you for your patience

mateoetchepare avatar Apr 09 '24 04:04 mateoetchepare

@mateoetchepare I also had some issues on windows, so Debian VM works ? That's good, I tried using github codespaces they also help because I no longer run into the Windows specific errors

monacodelisa avatar Apr 09 '24 04:04 monacodelisa

yes, in debian everything is working fine right now. in windows i tried different versions in those specific packages and i couldn't fix the error

mateoetchepare avatar Apr 09 '24 04:04 mateoetchepare

@goetzrobin I believe this issue should be closed :)

BoogieMonsta avatar May 17 '24 09:05 BoogieMonsta