Updated information about Clustered Lighting on Wiki #131
Conversation
JJL772
left a comment
JJL772
left a comment
There was a problem hiding this comment.
Page content needs some work. The screenshots you provided a pretty great though!
Updated the requested phrases, fixed minor spelling mistakes and compressed 2 images.
|
@JJL772 done |
me when I accidentally tab
There was a problem hiding this comment.
You have used a lot of images, like a lot. You shouldn't that many IMO and just instruct the user correctly where they need to go. The screenshots showing the flags menu with the effect it has in the game is a nice thing, though instead of having two separate images, basically condensing and reducing the amount of images used should be your goal.
Other than that, there are formatting fixes needed and few other bits so just look at the reviews.
|
@OrsellGit I don't mind the images. For tutorial-like stuff I prefer it actually. We do need a page with comprehensive technical documentation though. In the future, I intend to work on some of that. |
Updated styling, removed 6 images, reverted changes made to `test\_meta.json`.
OrsellGit
left a comment
OrsellGit
left a comment
There was a problem hiding this comment.
Everything looks good! Good job!
OrsellGit
dismissed
JJL772’s stale review
Dealt with review for Laveig for JJ, nothing too big changed from last review besides the requested changes.
3 participants
Clustered Lighting info hasn't been updated since the pages were initially added. This update completely rewrites them and adds an additional page about Clustered Light Inspector. Images have been updated as well.
Changes:
light_type_reference.mdanddifferences.md