Page MenuHome

Some references fixes
AbandonedPublic

Authored by Campbell Barton (campbellbarton) on Feb 3 2015, 11:44 PM.

Details

Summary

Here is some patch fixing bad terms or reference usage...
I don't see any target / index guidelines, so I felt free to use a couple of targets.

Diff Detail

Event Timeline

Fabien Devaux (fdev31) retitled this revision from to Some references fixes.
Fabien Devaux (fdev31) updated this object.

generally fine, noted some minor changes.

Presets could use general description.

glossary/index.rst
434–445

This goes against:

https://www.blender.org/manual/about/writing_style_guide.html#content-guidelines

Avoid enumerating similar options, such as listing every preset or every frame-rate in a drop-down.

Their contents may be summarized or simply omitted.

Such lists are only showing what is already obvious in the interface and end up being a lot of text to read & maintain.


In this case just explain in general terms what they do is enough IMHO.

modifiers/deform/lattice.rst
2

Would prefer these be a bit more unique _modifier-lattice for eg.

Campbell Barton (campbellbarton) requested changes to this revision.Feb 4 2015, 10:48 AM
Campbell Barton (campbellbarton) edited edge metadata.
This revision now requires changes to proceed.Feb 4 2015, 10:48 AM

(You may have guessed, but I didn't find a way to edit the patch :P)

modifiers/deform/lattice.rst
2

I think we can just drop it, I ended with a :doc: instead of a :ref:...

Same for the Video presets, maybe just leave the glossary entry, I'll remove the :term: later.

Fabien Devaux (fdev31) edited edge metadata.

I simplified the commit / updated according to latest version