Move advanced import settings to their own page #11127
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
skyace65
force-pushed
the
Advanced-Import-Settings
branch
from
588d5f9 to
b208565
Compare
skyace65
added
enhancement
area:manual
Piralein
requested changes
Jul 20, 2025
tutorials/assets_pipeline/importing_3d_scenes/advanced_import_settings.rst
Outdated
Show resolved
Hide resolved
tutorials/assets_pipeline/importing_3d_scenes/advanced_import_settings.rst
Outdated
Show resolved
Hide resolved
skyace65
force-pushed
the
Advanced-Import-Settings
branch
from
b208565 to
eae86b8
Compare
|
Fixed. |
Piralein
approved these changes
Jul 21, 2025
skyace65
force-pushed
the
Advanced-Import-Settings
branch
from
eae86b8 to
761caac
Compare
mhilbrunner
approved these changes
Jul 24, 2025
Calinou
reviewed
Jul 24, 2025
tutorials/assets_pipeline/importing_3d_scenes/advanced_import_settings.rst
Outdated
Show resolved
Hide resolved
skyace65
force-pushed
the
Advanced-Import-Settings
branch
from
761caac to
c7c8d75
Compare
|
Fixed |
Calinou
approved these changes
Jul 25, 2025
|
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
More or less does what the title says.
Given the amount of text currently on the import settings page I think it makes sense to put it on it's own page. For one it makes the existing page more clear. I was going crazy trying to figure out where the slices options are while reviewing another PR, only to figure out that it's only an option for animations in the advanced import settings. It's also worth noting that the advanced settings are underdocumented, there should be more text than there is now.
I'm planning on doing more work on this in the future to reformat how information is presented, as well as fill in some missing information, but I figured it made more sense to just do the split first.
The only text I've added on the new page is the introduction, as well as the text under the animation options section.
I've also renamed the "Storage" section for animations to "Save to File" since that's what the setting is named.