Comment

Comment by hailpixel

Very nice job. I like the feeling you are going for and the illustrations. I've marked-up this image ( http://i.imgur.com/HWiw7.jpg ) with some suggestions and things I would cleanup. Notes transcribed below. Hope these help!

1) Center the headshot vertically with the bubble.

2) Don't use an image to round corners, use CSS3 property border-radius instead.

3) This font is unique to this callout, use the Amethysta font face used on the rest of the page.

4) This isn't centered on my screen. Use font-align to center this text, not a static margin.

5) Using a large typeface for the first line/paragraph of text works well to call attention to detail. Make the following paragraphs the same font size.

6) The background colors of the images and the box they sit in are all slightly different.

7) I wasn't sure if this title was supposed to be sitting on the right margin. Centering it would work better I think.

8) These are links, but do no look like links. Use color to show they are clickable.

edit: spelling, grammer.

parent hailpixel 14 years ago

Discussion

Live from the official Hacker News API

A modern reader for Hacker News · SvelteKit + Skeleton