@Alex-Archer-I
Posted
Hi!
I love that you though about dynamic addon to urls! If I can understand correctly, it's purpose to track from which page user came?
Also you did cool thing with animation! By the way, there are useful method which can helps with it.
Literally couple of suggestions:
var
is rarely used in JS now, it mostly replaces bylet
andconst
.- Guess you should add
cursor: pointer
and some hover effect to button for better UX
Hope that helps =)
Marked as helpful
@dquinn089
Posted
@Alex-Archer-I
So I actually realized when I read this comment that I forgot to replace my currentUrl
variable placeholder with the live server url for the project. It seems to be working now(although the link worked in the share window I didn't actually share it, I don't use these websites lol).
Also thank you for the other suggestions, I feel the designed feels much more complete now. Good advice as always!
@Alex-Archer-I
Posted
@dquinn089
Oh, yeah, it's cos it's a share button! I didn't even think about it! Now I get why you need a dynamic url!
Haha, that's actually cool, that why I love this place, you look at someone's works and realized a huge gaps in you knowledge =)
And hey, you're welcome as always! Cool hover effect you did =)