DavidanKord10
@DavidanKord10All comments
- @tomxydSubmitted about 1 month ago@DavidanKord10Posted about 1 month ago
The colors are way off, and it's too small in height. For the color part there is a style file in the initial files that the site gave us, you can see that file for the color.
0 - @PranavReddy2703Submitted about 2 months ago@DavidanKord10Posted about 2 months ago
I think you should add a css reset code and you are using too px in width and height, this can compromise the flexibility and accessibility of the page
0 - @arolikSubmitted about 2 months ago@DavidanKord10Posted about 2 months ago
Well, the first thing I can see is that you made the whole box too small. Aside from that I can't find any more mistakes, good job. (I'm a newbie myself so take little consideration of this feedback :c )
0 - @rishavdhungelSubmitted about 2 months ago@DavidanKord10Posted about 2 months ago
I think you should separate the style part in a css file, as for the rest I think you did a good job by minimizing the content and still having the desired page. The code is well structured and readable, though I still prefer that you separate the style. The solution is a bit different from the design but it's still good.
1