Skip to content

Rename "Parallax corrected Cubemaps" to "General" - #214

Merged
OrsellGit merged 2 commits into
StrataSource:mainfrom
kiddu25:rename-pcc-page2
Aug 22, 2026
Merged

Rename "Parallax corrected Cubemaps" to "General"#214
OrsellGit merged 2 commits into
StrataSource:mainfrom
kiddu25:rename-pcc-page2

Conversation

@kiddu25

@kiddu25 kiddu25 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@OrsellGit OrsellGit left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just fix the title capitalization then this is good to go.

Comment thread docs/graphics/general/meta.json Outdated
@@ -1,4 +1,4 @@
{
"title": "Parallax Corrected Cubemaps",
"title": "general",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be "General", but not the folder, just the title.

@kiddu25

kiddu25 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

should be done

@ozxybox

ozxybox commented Aug 21, 2026

Copy link
Copy Markdown
Member

Is this PR correct? The diff looks odd, where it shows a bunch of files removed. Was the goal to remove these or move them elsewhere?

@OrsellGit

Copy link
Copy Markdown
Contributor

Yeah I am confused, all that was needed was the title casing to be fixed. I dont know why this other commit was done.

@kiddu25

kiddu25 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

the commit remove the PCC tutorial in the materials page

i remove it because there already a recent pr (that was merge) that add it into the graphics(previously call lightings) tabs

but i guess i can revert it if it's problematic it's just to avoid do to another PR for this

@OrsellGit
OrsellGit merged commit 6d96ca9 into StrataSource:main Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants