Skip to content

Add blender UI#3

Open
Carnage-Zore wants to merge 3 commits intoazimuthdeveloper:mainfrom
Carnage-Zore:Add-Blender-UI
Open

Add blender UI#3
Carnage-Zore wants to merge 3 commits intoazimuthdeveloper:mainfrom
Carnage-Zore:Add-Blender-UI

Conversation

@Carnage-Zore
Copy link

Hi, I saw this and got carried away after reading the readme and thought, I could add UI to this... So i did.
I removed the Export on Save and replaced it with a UI menu in the 'N' menu along the right hand side in a category called 'Export Curves'.

I moved the register functions to the end of the file before the name check.

1 Panel: ExportCurvesUI
2 Operators: ExportAllCurves and ExportSelectedCurves, these do what they say.
2 Properties in the Scene: curve_export_filename, curve_export_filepath.

The properties are for the exporting location and name

I modified one function to suit the needs of the new buttons but most of the code in the other functions remain the same.

Thanks
-Zore

Blender - Added UI(Available in the 'N' menu under 'Export Curves'), Moved the register functions to the end of the file, Removed the Export on Save Functionality in favor of Buttons, Added buttons to Export All or Selected Curves. (More Info in the file)
"Blender - Added UI(Available in the 'N' menu under 'Export Curves'), Moved the register functions to the end of the file, Removed the Export on Save Functionality in favor of Buttons, Added buttons to Export All or Selected Curves. (More Info in the file)"
@Carnage-Zore
Copy link
Author

I should probably mention that I tested this with Blender 5.0.1

@azimuthdeveloper
Copy link
Owner

You're my hero. I'll take a look at it, thank you

Copy link
Owner

@azimuthdeveloper azimuthdeveloper left a comment

Choose a reason for hiding this comment

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

logic looks good, i'm supportive of the print statements so people know what is going on. just a couple of typos and then we should be able to merge. thanks!

print('Folder does not exist.')
return False
if filename == "":
print('File name must be at least 1 charcter long.')
Copy link
Owner

Choose a reason for hiding this comment

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

just a small typo here

if filename == "":
print('File name must be at least 1 charcter long.')
return False
print('File is being saved!!')
Copy link
Owner

Choose a reason for hiding this comment

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

would it be beneficial to include the path of where the file is being saved to? also this would appear in the blender console correct?

@Carnage-Zore
Copy link
Author

I fixed the typos and added the output file path to the print.

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.

2 participants