Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding new "Choose Profile" context menu to Project Files #5634

Merged
merged 15 commits into from
Oct 12, 2024

Conversation

jonoomph
Copy link
Member

@jonoomph jonoomph commented Oct 8, 2024

Adding new "Choose Profile" context menu to Project Files, to make it easy to edit using your source file width+height+FPS profile. Large refactor of how profile switching happens (moving to UpdateManger - so it will support undo/redo system). Add new profile() method to File Query class, to make it easy to generate/find a profile object for any File object.

Plus... avoid error message prompts when importing multiple files.

image

image

… easy to edit using your source file width+height+FPS profile. Avoid error message prompts when importing multiple files. Large refactor of how profile switching happens (moving to UpdateManger - so it will support undo/redo system). Add new profile() method to File Query class, to make it easy to generate/find a profile object for any File object.
Copy link

sentry-io bot commented Oct 8, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: src/windows/models/files_model.py

Function Unhandled Issue
add_files NameError: name 'elements' is not defined windows...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

@jonoomph
Copy link
Member Author

jonoomph commented Oct 8, 2024

…e, and Delete custom profiles. Updated translations (new strings). Modified "Choose Profile" to apply a new profile, even if it's already applied (i.e. might have been edited now). Added new methods for inserting / updating / removing rows from the Profile model / view. Fixed small bug on the Export screen, to restore start/end frame correctly.
…Delete) options - to the user guide documentation.
…en adding a new custom profile and the context menu not updating correctly. Also, fixed the Profile comparison to use "is" since we just modified the Swig bindings for Profile equality operators.
…ch is currently used or the default project (in settings). Also validating custom profile has a unique description. Adding icons to Profile context menu. Added a new "Set as Default Profile" context menu option. Updated translations.
… selection when calling self.select_profile()
@jonoomph jonoomph merged commit 5ed8300 into develop Oct 12, 2024
9 checks passed
@jonoomph jonoomph deleted the choose-profile-v2 branch October 12, 2024 20:43
Copy link

sentry-io bot commented Oct 15, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ ZeroDivisionError: division by zero windows.models.profiles_model in update_model View Issue

Did you find this useful? React with a 👍 or 👎

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.

1 participant