URL shortening API landing page

Solution retrospective
Hello, Frontend Mentor community! This is my solution to the URL shortening API landing page.
I appreciate all the feedback you left that helped me to improve this project. I fixed the issues I had previously and added new features per your recommendation.
I have done this project a while ago however if you are here and have any feedback, tips, or ideas to share, I will be more than glad to hear them out!
Thanks
Please log in to post a comment
Log in with GitHubCommunity feedback
- @omarelbastawesy
Great job :)
- @Luis15Herr
Amazing Work :)
- @TSune-web
Well done!
- @shrki416
@catherineisonline great job! This looks awesome :)
- @NicolasGula
Awesome!
- @jgengo-alt
looks very good, your logo (the brushes) are a bit disappearing compared to what is expected, but that's just a detailed. It's because you used the img to also do the background-color and the border-radius: 50% instead of inserting the image in a div with those ;)
you did very great work here! thanks for sharing
- @msuryaditriputraR
Awesome Work..
- @AditNovadianto
@catherineisonline you did this project so nice and not have any issues there!!!
- @BikeInMan
If @nakoyawilson 's solution works for you, then please ignore. Unlike her, I didn't get time to clone and test it.
But I get a feeling,
mainClone.classList = "search-result";
is wrong.May be it should be written as
mainClone.className = "search-result";
ormainClone.classList.add('search-result');
Just a thought.
BTW, you always say, it is not about design and just practicing js, but most of your work seems to be spot on. Great work as always.
- @nakoyawilson
Hey Catherine! I downloaded your code, changed all occurrences of
sessionStorage
tolocalStorage
and deletedoriginalHtml = localStorage.getItem("cloneCache");
andparentNode.innerHTML = originalHtml;
from the function and the data displayed for me.window.onload = function () { let mainClone = shortlyResult.cloneNode(true); let cloneLinkField = mainClone.querySelector(".inserted-link"); mainClone.classList = "search-result"; parentNode.appendChild(mainClone); //Retrieving let cloneLink = localStorage.getItem("cloneLink"); console.log(cloneLink); //Injecting console.log(cloneLinkField); cloneLinkField.innerHTML = cloneLink; };
- @Ken9Kodehode
Great work!
- @kerminek
Hi, one image makes your page wider than we want it to be. Solve it using "overflow: hidden" on main container of the page. Also, if you want to keep data after refresh try to make it using cookies or simply "localstorage" :)
- @GitHub-dev12345
How to make this project and where are you learn APIs. Can you share how to learn APIs.
- @GitHub-dev12345
Hey Catherine nice work😊🎉
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord