Fix T46078 Rework Smoke Materials. I also cleaned the BI volume page. i think this is the only image that is not included in the diff
Details
- Reviewers
Kitt Zwovic (gandalf3) - Group Reviewers
Documentation - Maniphest Tasks
- T46078: Smoke material should be split up into cycles and BI
Diff Detail
- Repository
- rBM Blender Manual
Event Timeline
Nice work!
This looks pretty good to me, but couldn't get the diff to apply properly here.. Are you able to apply it on your end?
And a minor nitpick, but I noticed a lot of phrasings sound rather "instructiony"; like "go here, select it, then give it this". Not a huge deal, but I feel it doesn't quite fit with the reference manual goal (sounds more tutorial-like to me).
It may because the images are empty. I will remove them from the patch and when the patch is committed I will remove and add the images
It's still only partially applying to volume.rst
Are you creating the diff against an up-to-date working copy?
I tried to redo the patch from scratch. but still does not apply. But im successfully building my local repository.
It seams that non of my patch will apply. Something must be wrong with the .diff file. because my commits are fine so I will commit this when it is ready to be merged
It *looks* fine from what I can see in the diff, but I can't check for compile issues etc. without applying the diff.
Not too sure what's to be done in such situations.. If you can commit it okay then perhaps that will do
