The-King-of-Toasters / henry-george-jr_the-life-of-henry-george

Other
0 stars 0 forks source link

Section 1 Review #1

Open The-King-of-Toasters opened 4 years ago

The-King-of-Toasters commented 4 years ago

Section 1 is completed and ready for review. This issue is essentially a "code review" and discussion of problems I've encountered.

Linting

CSS

Letters

Semantics

... More to come

acabal commented 4 years ago

I'm not familiar with raw images or xmp files. However if a file can only be read by Darktable, then we should replace it with an open format or remove it, because not everyone is going to have purchased that software. Also you can remove .orig from the filenames.

Yes, remove unused CSS selectors.

If you mean nbsps before ampersands, check the typography manual. In cases where there are company names like "Hawkins & Harker" then we want an nbsp before the ampersand.

Please use .signature for now as that's the style in the manual. Later we will update it to z3998 but the entire corpus must be updated.

You can add padding to datelines but leave the font size the same.

In terms of semantics, my rule of thumb is that if we're using a tag already (like <i> or <blockquote>) then add semantics. Otherwise as you noticed, it's possible to go down a very deep rabbit hole. However, the book should be consistent in semantic use. If you've already started then it either has to be finished in that style, or we have to go back and remove semantics to make things consistent across the whole book.