Anurag Mohanty• 160
@ar-mohanty
Submitted
Arlagonix
@arlagonix
All comments
- Arlagonix• 410
@arlagonix
Posted
Woah, it looks awesome! No problems whatsoever
0 - Glozsa• 180
@Glozsa
Submitted
Nice 20 mins practice
Arlagonix• 410@arlagonix
Posted
Nice solution (I think)
Consider adding draggable="false" to the img so people wouldn't accidentally drag it around
Consider reducing space between header and paragraph. Closely related things must be close to each other. Presently the header is closer to the image than to the paragraph
The card isn't responsive when the screen width is less than 343px. Actually there are usually no screens with such small width, but still
Btw awesome shadow
Overall great solution
Marked as helpful
1 @alawi17
Submitted
How did you remove scrolling? Can someone help me with my code. Thanks in advance 🙂
Arlagonix• 410@arlagonix
Posted
How did you remove scrolling? There are 2 options:
- Set margin: 0 to body tag. I recommend this approach
- Set overflow: hidden to body tag
Also your background doesn't stretch to full width
If you're publishing your code on Github, I recommend to use Github Pages to host there your working application. Thus you will keep both code and demonstration in one place
Marked as helpful
1- Diogo Quina• 30
@DQuina
Submitted
Aceito opiniões sobre o meu segundo projeto realizado através de programação HTML / CSS : )
Arlagonix• 410@arlagonix
Posted
Looks awesome! Woah, you absolutely nailed it!
1 - Lucas Matheus• 400
@Lukiticas
Submitted
I missed so much you guys <3 but i'm back, trying to get on the rails again.
Soooo this project was quite a journey i'd say, i'd learned a lot about custom libraries. Managing state with react still is an issue for me so i would appreciate any advice or recommendations about it!
I'm planning to implement a couple more things, right now i'm using GNews API to get news about the country but they have a request quota and the hosted images are often undefined, do you guys know about an api for this kind of thing?
Anyway, hope you liked it, sorry for the bad English and much love <3
Arlagonix• 410@arlagonix
Posted
I don't know how to ignore the errors, but the solution looks great!
- Smooth animations
- A map
- A list of related news articles
- Loader showing when the page is loading
Splendid!
1 - Fat• 910
@Fahatmah
Submitted
Hey Guys!
Just finished this accordion and learned a bit of Javascript. But you can also make an accordion without Javascript.
I need some feedback on my other solutions so I can learn from it.
Learning is ENDLESS!
Arlagonix• 410@arlagonix
Posted
It looks awesome! I love it!
-
The layout is responsive and looks nice on different resolutions. Although on the smallest resolution text inside of the 1-st, 3-rd sections seem to not have enough space
-
Awesome animations! Looks very smooth. It looks fantastic that the background also resizes with the frame when you open the details
-
Nice that you made the bold highlight only when the details are opened. It looks better than in the initial design
-
It seems to me that the font size might be a bit bigger. 11px and 14px are a bit small. There is enough space for bigger text: 16px + 24px for example
-
I think it would look better if your stylized a bit more the part "Challenge by ... Coded by ...". It looks a bit out of place. I'd make the text bigger, changed the font family, the font color
-
Also changing gradient background might fit here perfectly (as soon as it's initially already some sort of gradient). Here's a guide from Kevin Powell channel: https://www.youtube.com/watch?v=f3mwKLXpOLk&ab_channel=KevinPowell. It's quite easy: make background size big and move it from side to side with an infinite animation
-
Another way to enhance the solution is to probably add some icons to the left of the questions. There are not so many of them, but that might make the text easier to look through
Marked as helpful
1 -
