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

[Feature request] Add 'Deselect All' button to the Hierarchy #1

Closed
HeCorr opened this issue Feb 10, 2023 · 10 comments
Closed

[Feature request] Add 'Deselect All' button to the Hierarchy #1

HeCorr opened this issue Feb 10, 2023 · 10 comments
Labels
enhancement New feature or request

Comments

@HeCorr
Copy link

HeCorr commented Feb 10, 2023

(somewhat copied from Perfare#961)

Hello!

I would like to request a feature that would allow me to de-select all checkboxes in the hierarchy because due to my current workflow I use "Go to scene hierarchy" from the Asset list all the time and when I need to export the next asset, the previous one is already scrolled away from the hierarchy view, forcing me to find it again just to de-select it.

Another option would be a setting that automatically de-selects all previously selected assets after exporting.

Thanks!

@aelurum aelurum added the enhancement New feature or request label Feb 21, 2023
aelurum added a commit that referenced this issue Mar 11, 2023
- Added context menu with "Select all", "Select none", "Expand all", "Collapse all" options to the "Scene Hierarchy" tab.
#1
@aelurum
Copy link
Owner

aelurum commented Mar 11, 2023

Hi
You can try it now by downloading one of the versions from appveyor.
For example: AssetStudioModGUI-net6-win64
I added some options to a right-click menu, but keep in mind that depending on the number of nodes in the "Scene Hierarchy" tab, it may be slow.
As for "automatically de-selects", well I'll think about that a bit later.

@HeCorr
Copy link
Author

HeCorr commented Mar 11, 2023

thank you very much!

@HeCorr
Copy link
Author

HeCorr commented Mar 11, 2023

it doesn't seem to be launching when I click it.

@aelurum
Copy link
Owner

aelurum commented Mar 13, 2023

What exactly didn't launch?

@HeCorr
Copy link
Author

HeCorr commented Mar 13, 2023 via email

@aelurum
Copy link
Owner

aelurum commented Mar 13, 2023

I see
You likely need to update your .NET 6 Desktop Runtime to run this version
And you can also try the net472 version

@HeCorr
Copy link
Author

HeCorr commented Mar 13, 2023

thanks! that worked, but I'm still having the errors I was having with the original version, which is why I tried using this fork to begin with.

I'll open a separate issue to report them.

@aelurum aelurum closed this as completed Mar 14, 2023
@HeCorr
Copy link
Author

HeCorr commented Apr 20, 2023

Hello! So, indeed the de-selection performance is very slow with a large list so I was wondering if instead of looping through all the items it wouldn't be possible to just loop through the selected ones instead?

Surely the selection state is stored somewhere, which is how it knows which items to export, right? Or does the exporting code also loop through all the items in the list and exports the selected ones..?

@aelurum
Copy link
Owner

aelurum commented Apr 26, 2023

Or does the exporting code also loop through all the items in the list and exports the selected ones..?

Almost)
And they are recursive functions, so they are a bit slow.
But yeah, it's worth to try to speed up it then.

Here, try this: AssetStudioModGUI-net472 or all artifacts.

@aelurum aelurum reopened this Apr 26, 2023
@HeCorr
Copy link
Author

HeCorr commented Apr 26, 2023

There you go, it's basically instantaneous now. Thank you very much!

I think you can close it now :)

2023-04-26.15-00-04.mp4

@aelurum aelurum closed this as completed Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants