Thanks so much for these - I'm slowly working my way through them. The TR code has changed since I made the layout I think, which is the source of some of these things. And so much of this would have been REALLY useful to know when I made the layout.... only I made the layout for me, not anticipating I would do any of this, so I didn't pay any attention to making it more streamlined and compliant with any guidelines but my own!

I do have a couple of questions.

Regarding point 5, I understand the principle behind what you've said. However, in some cases my background colour is not a color_*_background, because I create the double border effect by switching the border and background colours first in the outermost element and then use the background colour as the background colour in the inner element. Like this:

.navigation {
background-color: $*color_module_border;
border: .5em solid $*color_module_background;
color: $*color_module_text;
}

.navigation ul {
background-color: $*color_module_background;
}

Is that likely to cause problems? Do I need to create more colour properties for this instead of handling it as I do?

And do I set the text colour in both of these, or is it sufficient to set it in .navigation as .navigation ul will inherit.

Thanks!