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
Implements 'trim-by-grid' CL option #2010
Conversation
else if (opt == &m_options.trimByGrid()) { | ||
cof.trimByGrid = true; | ||
if (m_exporter) | ||
m_exporter->setTrimByGrid(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we talked in the email, I think here we can call m_exporter->setTrimCels(true);
too.
Please try to use commit -amend and push -force to make easier the merge and avoid a lot of little commits. Please also give you a time for testing. |
b761e3f
to
5e132c4
Compare
@@ -37,6 +37,7 @@ namespace app { | |||
bool listSlices; | |||
bool ignoreEmpty; | |||
bool trim; | |||
bool trimByGrid; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
About this field you have two options:
- Remove it from this PR (because it's not used)
- Or complete this PR with the trim-by-grid support for
--save-as
…rim-by-grid-CLI # Conflicts: # src/app/cli/cli_processor.cpp
aseprite --trim-by-grid --sheet
(#1926 )