It’s not necessarily 'bad' to use 'BLOCK_' it is just more challenging to figure out what the class is trying to do by just reading the name. It is 'better' to give it a descriptive name - it makes it much easier when you are editing your css for style changes. For example:
Code:
Instead of: Use:
block_1 title-img (and no nead to define a language(French) for an image file…)
block_2 author
block_3 publisher
Don’t use <body class=“calibre">, define a 'minimal' body style (only things you absolutely MUST have - I don’t use anything) in your css like this: body {blah blah blah}
I wouldn’t define line-height at all, and certainly never use a fixed size (points - pt) to define margins. Use a relative size, like em so that it scales appropriately with the font size the user chooses.
Likewise I’d be very careful when you use letter spacing. Just use a good font and rely on the font’s properties. If you do use it, again use em instead of pt otherwise it is useless when the user changes font size.
I can guarantee others will pop in with their recommendations - just make sure you try hard to differentiate between real problems and just personal preference.
Cheers!