Second time working with an API. I learnt some new things and got it to work, but I can't catch any errors for some reason. Please, feel free to suggest ways I can correct the problem.
Mofe
@Mofe620All comments
- @CodeyenumSubmitted over 2 years ago
- @CodeyenumSubmitted over 2 years ago
Second time working with an API. I learnt some new things and got it to work, but I can't catch any errors for some reason. Please, feel free to suggest ways I can correct the problem.
@Mofe620Posted over 2 years agoThis is really nice, Olise. I had difficulty implementing the loading animation using plain CSS and Vanilla JS. How did you do it ?
0 - @Da-Gu-Gu-GuSubmitted over 2 years ago
- @CuencaGITSubmitted over 2 years ago
I kinda struggled a little bit in implementing my mobile version into the desktop. I just manually re-arrange all of the components using position: absolute; can you guys suggest how to do it more easily.
@Mofe620Posted over 2 years agoWrap your
h2
andp
tags in a div Create a container div around everything in yourmain
section, apart from the nav. Set the container to a display of flex in the desktop mode. See if it works.1 - @deksa89Submitted over 2 years ago
To complete this task I've used combination of grid and flex as well as scss to automate some adjustments. Any feedbacks about best practices are welcome. :D
@Mofe620Posted over 2 years agoYou have a lot of text edging out in wider resolutions. You may want to set
overflow-y:hidden;
on your cards or reduce the text. I don't think the challenge has that much text on all the cards.Cheers 🥂
1 - @alphiechadoSubmitted over 2 years ago
What did you find difficult while building the project?
@Mofe620Posted over 2 years agoWell done, Alfred Just a few points to note:
- I don't think you really need those pixel values for top and left. Your
justify-content
andalign-items
should take care of that. - A little padding on your
p.para2
should make things look a little nicer.
Cheers 🥂
0 - I don't think you really need those pixel values for top and left. Your
- @abeera17Submitted over 2 years ago@Mofe620Posted over 2 years ago
Great job, Bari. To center your component, you can give the
body{ display: flex;}
and justify content, and align items to center.You're missing the language attribute in your opening html tag. It's important for accessibility purposes.
<html lang="en">
.On a whole, you've done well ✅
Marked as helpful0 - @christian-prasetyaSubmitted over 2 years ago
The most difficult of this project is:
- Creating the color overlay on the image
- Change the image source based on the size of the page
- Maximizing the responsive layout to create a good page on the right size
Hopefully I done good job on this project? Or maybe have a better solution?
Please give me feedback?
Thank you
@Mofe620Posted over 2 years agoGood job, Christian
Your image isn't being displayed. You may want to take a look at your src attribute.
As for the blending,
mix-blend-mode: multiply;
does help.0 - @Deevyne99Submitted over 2 years ago
Hi just finished my grid challenge at the end of this challange i learnt responsiveness it was quite challenging at first but i scaled through .
@Mofe620Posted over 2 years agoWell done, Kachi. Just take note of the accessibility issues raised in the solution report.
0 - @devmor-jSubmitted over 2 years ago
Disabled elements can not emit events and that make it hard to toast notification if user clicks during API waiting time (2 sec). So this was a valuable lesson and I tried to come up with a solution using CSS only. Greyed out
box-shadow
will inform user of the waiting timeout.@Mofe620Posted over 2 years agoThis is amazing 🤩🔥 The spin on the Dice 👏👏 The transitions too. Well done ✅
1 - @Bube-createSubmitted over 2 years ago@Mofe620Posted over 2 years ago
Hello 'Bube 😀 Your work looks good. Just a few points to note:
- You didn't import the 'Manrope' font family
- For your Dice, you could have used a
<div>
instead of a<button>
since buttons need to contain text.
Overall your work looks good and works fine. Your GitHub repo and personal site is nice as well 👏
0 - @champer3Submitted over 2 years ago
How do I make sure that I can only click on one of the rating at a time
@Mofe620Posted over 2 years agoHey Stephen. Great job there. I would do that using JavaScript. Get all 5 areas in an array and allow the selected iteration to be active while set the others to inactive using the
array.filter()
method.Your work looks good. Just remember to include
lang="en"
within your<html>
tag for accessibility purposes. Cheers 🥂2