352written posts
offlinecurrently
hi j. !
This template seriously looks so good! On its overall harmony, the template is a very pleasant sight. Everything is in synergy and nothing stick out badly. That means the way you're hightlighting certain parts (like the lyrics and the image) FITS. Despite the small padding, your usage of gray text was smart because the contrast alone is enough for spacing without sacrificing readability. Your judgements are really good!
You highlighted the image with a lighter border-color and that's a really nice decision. The hover, however, is lacking. Hovers are used to "sate the curiosity" of being drawn into that visual piece. Your cues were nice, but the contents were underwhelming. I would suggest that you use the cursive font on words and have the word count as a sans-serif (or vice versa). Play with your typography & colors on that part. Also try making it more into the middle of the image, rather than the bottom (a nitpick, but i think the current position is also nice).
The minor problem with your current template is that everything blends so well that it doesn't give any artistry for you. I did say that nothing sticked out too badly, but nothing popped out too. The way everything is aligned perfect with each other made it flow too smoothly. It's a safe template to make for the first time, but I suggest to start crazing up now (while also minding design guidelines- because trust me they help). Make it flare. Add something that would make them go thru pages in their history to find your template! It could be your color, layout, or even your font usage. Small things make big changes. If you really want to improve your aesthetics, go wild for now. Your blending skills will come in handy later when you make skins with matching templates. I swear, it's a nice skill to have, but it will hinder your current objective. For starters, try to pick atleast one/two part of the template that you want "designed". Either it's on side, top, or bottom. Also play with the flow through paddings, margins, borders, colors, etc. It'll make your future templates more dynamic. Overall, this is a really nice template to see. For you coding, it's neat to look at. There are some instances tho that you forget to add px/units. Height auto on #jim is unnecessary, you can omit it. You can achieve a neater code without using positions. Don't try to abuse positions, they could be a bad habit after all as it can be buggy. Especially when they are substituted for tables. ".jim-title .jim-title2" you dont need to call out parent if the child have unique class. It's fine if it's just .jim-title2.
|
last edit on Sept 20, 2018 17:17:31 GMT by von
|