-
Notifications
You must be signed in to change notification settings - Fork 43
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
save-export-clean should flatten #4
Comments
Somehow I believe it's better not to flatten. If you've layers and you save to a non-XCF format, that's an error. Now I believe it would be better to just prompt a notice dialog, rather than a Accept/Cancel dialog. For example, if you save a layered image to non-XCF, I believe it would be wise to just display a notice like this: "You saved a layered image in a format which doesn't support layers. Only the current layer was saved." I don't think such notice is an annoyance, because it's only displayed if you forget to flatten, and the user is expected to not forget to flatten (at least I'm the kind of user who prefers non-automagical software, so I wouldn't like to have a silent flattening without my consent :-) (Btw, I'm assuming it saves all layers when you save to XCF, but I'm not sure, since the call to the save function has a drawable argument anyway, so I don't know if such drawable argument is ignored when saving to XCF, or if you've to use another function for saving to XCF) |
Let me just add a great big thanks! |
Some time ago l avoided updating my gimp version from 2.6 to 2.8 mainly because l used gimp to edit textures and it was very useful that l only needed to open one, draw a few lines, hit ctrl+s and close to edit it. But then l switched to 2.8 because G'MIC didn't longer work right l think. |
The page linked off the README describes the difference between the two. |
So Saver is newer than Save/Export clean? |
Yes. I thought that was clear from "But I found that I wasn't using Save/export clean much myself, and eventually, I wrote a more general plug-in called Saver." Is there a way I could rephrase that to make it more clear? |
in the README the save-export-clean.py has this in its description |
I've changed it to this: save-export-clean.py Is that clearer? |
yes, thanks |
Several people have commented that save-export-clean saves only the current layer. If there are multiple layers and we're saving to a format other than XCF, we should flatten. Preferably without pestering the user with dialogs, because if we're saving to jpg, of course we want to flatten, duh!
The text was updated successfully, but these errors were encountered: