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

Export dialog UX overhaul #781

Merged

Conversation

OverloadedOrama
Copy link
Member

@OverloadedOrama OverloadedOrama commented Nov 21, 2022

Implements #605 (comment). Deviates from the original proposal and does not implement all of the new suggested features yet.

image

This PR makes the export dialog more intuitive and simpler to use by merging the "Frame" and "Animation" tabs into one. Now, all frames will be exported as multiple png files by default, with the ability to choose only to export the currently selected frame(s), thus keeping the functionality of the previous "Frame" tab and only exporting one png file, or multiples if there are more than one selected frames. To export a .gif or a .apng file, simply change the file extension.

image

The PR also adds some new features:

  • Specific layer exporting. Choose between all visible layers, selected layers, or specific layers. Also supports exporting group layers that blend their child layers. Kind of implements Exporting layers as seperate images #773.
  • A new CollapsibleContainer custom node, which is being used to hide advanced settings. This node can be used anywhere in the UI, not just the export dialog.
  • Specific frame exporting for multiple pngs, gifs and apngs, besides just spritesheets which is the case in the current master branch. Should fully implement Label-based animation export #472.
  • Animation direction now supports multiple pngs and spritesheets besides just gifs and apngs.

Future improvements that can be made but are outside the scope of this PR:

The PR should be ready to get merged, but feedback on the UX as well as testing would be very welcome!

@OverloadedOrama
Copy link
Member Author

OverloadedOrama commented Nov 26, 2022

Updated the PR to add a new CollapsibleContainer custom node, that is being used to contain the advanced settings. I also added a VSplitContainer between the preview and the options.

@OverloadedOrama OverloadedOrama merged commit 4d113d3 into Orama-Interactive:master Nov 28, 2022
@OverloadedOrama OverloadedOrama deleted the new-export-dialog branch December 16, 2022 22:50
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