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

Praise: This is awesome #27

Closed
bradydowling opened this issue Oct 2, 2014 · 9 comments
Closed

Praise: This is awesome #27

bradydowling opened this issue Oct 2, 2014 · 9 comments

Comments

@bradydowling
Copy link

I've never given feedback to someone on Github before so I'm not sure if this is the proper method but this script was a lifesaver for me. It took forever for the built in PS script to run and it didn't even get me what I wanted. Your script is gold. Thank you very much.

@antipalindrome
Copy link
Owner

This area is typically to bring up bugs or suggest new features, but we appreciate your feedback Brady. Thanks! We also accept donations :)

@skjorn
Copy link
Collaborator

skjorn commented Oct 2, 2014

You are very welcome. I guess this is the best place for all kinds of feedback and yours is much appreciated. I'm glad we could help :-)

@skjorn
Copy link
Collaborator

skjorn commented Oct 2, 2014

I think we could keep this thread open if anyone else wants to drop a cheerful note. They can comment here.

@skjorn skjorn reopened this Oct 2, 2014
@stigzler
Copy link

stigzler commented Oct 6, 2014

Agreed + donation made! Good work chaps. :)

@skjorn
Copy link
Collaborator

skjorn commented Oct 7, 2014

@stigzler Thank you for both -- praise and donation ;-)

@Charlie37
Copy link

Awesome work by you guys. Thanks a lot !
Works much better than the PS script, that badly manages prefixes and suffixes too (that's how I stumbled upon your script).

A small feature I added, maybe it's very specific to my clients: all our layers are named from PNG images that were imported via the PS script > Load Files Into Stack.
When using your script, I don't want any extension to be added to the layer name, even though I selected the PNG export option.
So I cropped the filename in getUniqueFileName():

// Check if the layer name already contained the extension.
if( ( fileName.substr(fileName.length - ext.length) ) == ext )
    fileName = fileName.substring( 0, fileName.length - ext.length );

Cheers !

@skjorn
Copy link
Collaborator

skjorn commented Nov 21, 2014

@Charlie37 Hi, thanks. I updated the script to strip extensions from layer names by default (can be turned off if "Name files: As layers" is selected). Your version wouldn't work if importing and exporting in different formats (e.g. read JPG and save as PNG). I recommend to re-download the script.

@Charlie37
Copy link

@skjorn Hi, I've re-downloaded the script and that feature works pretty well. Thanks a lot !

@skjorn
Copy link
Collaborator

skjorn commented Nov 24, 2014

@Charlie37 You're welcome :-)

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

No branches or pull requests

5 participants