Page MenuHome

Update image_as_planes addon text
ClosedPublic

Authored by Hugo Schouppe (HUSCH) on Sun, May 17, 4:19 PM.

Details

Summary

The current documenation about the addon Images as planes contains several TODO's and is also missing an example picture. I've updated the TODO's and added an example of the properties panel of the addon.

Diff Detail

Repository
rBM Blender Manual

Event Timeline

Hugo Schouppe (HUSCH) requested review of this revision.Sun, May 17, 4:19 PM
Hugo Schouppe (HUSCH) created this revision.

Tobias Heinke (TobiasH) added inline comments.
images_as_planes.rst
23 ↗(On Diff #24842)

"basically", "simplifies", "nicely" is wordy
"on top of each other or spaced" don't duplicate the reference below.

27 ↗(On Diff #24842)

pls. move that under the sections: Shader, Position
"some choices ... concerning", 'of cause' is wordy
"finetune" is informal

29 ↗(On Diff #24842)

presets are doced in the Interface chapter, so not needed

44 ↗(On Diff #24842)

"rel. or abs. way" is vague: with a path.
"select" to 'enable/check' a checkbox (also below)

what an img sequence is explained in data>media>graphic, better just link there.
"number of frames" to frame range, but if the user has to do nothing/ expect nothing else then this can be omitted .

51 ↗(On Diff #24842)

Corner Pin can have a link, also other main subjects: principled, constraints (can be done later...)
upper case Compositor.

66 ↗(On Diff #24842)

no emphasis on node names.
the last e.g. should be 'and thus' or similar

99 ↗(On Diff #24842)

missing empty line, no indent

120 ↗(On Diff #24842)

no "available options" append the axis options like:
https://docs.blender.org/manual/en/dev/animation/constraints/relationship/shrinkwrap.html#projection
no "appears" add an sub definition also for options like fill/fit, "two extra options":

Camera Relative
   ...

   Fill
      ...
images_as_planes.rst
23 ↗(On Diff #24842)

Hi Tobias,
Thanks for your comments. I will try to integrate them in the new text.
But, I am new to this system. So, can I / should I do this inline in the text like you did with this feedback? Or should I create a complete new diff-file?
When do I have to check the checkbox Done?

Hello Hugo,
Please create a new diff-file and update this revision by
clicking the Update Diff button in the box on the right,
then the same steps as for the initial creation.
(on the first page, mind to add the manual as repository again else that gets removed)

When you think you fix it; after the update (I don't remember if that can be done at the end of updating).

Aaron Carlisle (Blendify) requested changes to this revision.Tue, May 19, 4:32 AM

Please update with the changes by @Tobias Heinke (TobiasH)

This revision now requires changes to proceed.Tue, May 19, 4:32 AM

The comments of Tobias are integrated.

Hugo Schouppe (HUSCH) marked 8 inline comments as done.Tue, May 19, 11:07 AM

Please create the diff from your project directory then it's easier to apply.

images_as_planes.rst
26 ↗(On Diff #24908)

Nodes sentence should be under Material Settings.
Preset under Properties.

98 ↗(On Diff #24908)

The offset to Offset Planes.

114 ↗(On Diff #24908)

missing empty line

133 ↗(On Diff #24908)

Main Axis and Face Camera should also be definitions:
first Main axis,
then 'Face Camera is similar to Main axis' the differences.
Z-, ..., X+ combined in a single def.: Global axis... and the Z+ example from above.

what the default options are don't needs to be mentioned.

Corrected the text according to the remarks;
Not sure about the requested blank lines.

This revision was not accepted when it landed; it landed in state Needs Review.Fri, May 22, 11:34 PM
This revision was automatically updated to reflect the committed changes.