that was pretty fun challenge
transform to an element its ::before and ::after will get a new Stacking Context which will make it appear on top of the element which is weird.if you have any suggestions please let me know 🐢
Yacine,
This is great work. I am very impressed by the additional animation that you added. it makes it very clear where your new link is located. Keep up the great work.
One thing that I would suggest is to add some sort of error when a person adds an invalid link or just some random text. at the moment your solution does not have that.
Other than that Amazing work.
Keep it up
hey , do you have any feedback ? if (yes) { comment(); }
Very good work Amjed, you did an amazing job with the responsiveness of this project. One thing that I did on the larger screens sizes was make the search/filter bar absolute so the cards don't get pushed down when it pops up. On mobile kept it normal because the filter tags have to wrap around and the div below needs to move when it grows.
Try that if you want. Again amazing work. Keep it up
Great work. What happened to the link shortening part? It is actually not so hard, try and do it with Axios for the api calls if you are using React. I did it and I learned a lot about api calls etc. The responsiveness is good. there are a couple parts where the cards get squeezed, but other than that keep up the good work!
This is amazing work. It is responsive which is great, I have seen a lot of projects that are not so responsive.
One small suggestion, I would make the "search/filter box" position absolute on bigger screens that way it wont push down the cards.
There is not much to do about the cards going down on smaller screens unless you give a big margin, but then the design does not look nice.
Thumbs up and great work.
Shalom Keep up the efforts.
I would use something like bootstrap if you have never used it before?
It will help you learn about responsiveness. I have learned a lot from it and I am now able to use CSS if necessary to make sites responsive. It will contain all the elements in there places if used correctly. here I see it was a bit of a struggle with the cards and parts of the design overlapping.
Keep up the good work. all the best
Hey great work,
There are a few things I would try and work on.
User should have the ability to shorten multiple links. Here you can only paste one at a time.
The copy button turns to copied but never goes back to copy. if you plan on having only one link shorten card, I would suggest that you change back the button once a new link is shortened.
The line behind the cards is actually not as hard as it looks. check my solution for ideas if you want - https://www.frontendmentor.io/solutions/responsive-link-shortening-page-css-html-js-bootstrap-axios-oRkOrRZaJg
Overall great work. Keep it up
What do you think about the navbar and its responsiveness?
Keep up the good work. One of the things I noticed is when you hover over the menu and the orange border bottom shows. the header moves. an easy fix for that is to make a transparent border bottom and just change it on hover.
Keep up the good work
Hello,
This was a great try. I am not sure if it is just my screens but II think the responsiveness of the website needs work.
One thing I noticed as well as when you click on the thumbnails The big Image gets switched to the url of the Thumbnail and it becomes blurry.
Try to change them to the regular image.
Overall great work. Your code is very clean. I looked at the JavaScript Thumbs up.
I don't know if I'm writing a good code all I know is my code is working. Please I will welcome any kind of feedback
Hey,
Great job. I agree with Jhon. great in mobile and Desktop, but the in-between sizes need work. I would suggest trying to get the images to be able to change on mobile with the arrows like the original design and when the pop up is open the big image is missing. I know it is not easy, but it is worth the effort. You will learn a lot from building the project as close as possible to what was given. Keep up the good work