Nguyễn Đức Chính• 330
@chinh1402
Posted
I agree with Reza Sajjadian comment, I'll provide my suggestion in addition to his comment:
- The spacing between Summary items is too large, It would look better if you shorten that spacing a bit. Because those items are similar to each other, should be positioned close to each other to form a group.
- Without box-shadow, the Summary part blended into white background, and that's bad practice. As far as I am concerned, box-shadow can't really be extracted just by looking at a jpeg image, if there's a design file, the job is much simpler. In this case (jpeg), you should choose one box-shadow that looks similar to the design file by searching for one on Google, I suggest cssscan
Marked as helpful
0
Levi Ribeiro• 140
@ribeiroLevi
Posted
@chinh1402 the box-shadow really is a question for me, because the div i used to contain the project - marked with the id = "container" - occupies 100% width and 100% height and i'm not sure how to make a div to contain ONLY the project, instead of the whole screen. Any tips? thanks for the advice! :)
0
Nguyễn Đức Chính• 330
@chinh1402
Posted
@ribeiroLevi My solution is to make another div inside the div with id = "content" like so:
<div id="container">
<div id="content">
<div id="score">...</div>
<div id="summary">...</div>
</div>
</div>
That way, the div with id = "content" will only wrap around the project instead of the whole screen, you can do box-shadow on that div
Marked as helpful
0
