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

Multiple Features - Implements old requests/issues #118

Merged
merged 24 commits into from
Sep 17, 2018

Conversation

Willena
Copy link
Contributor

@Willena Willena commented Sep 9, 2018

In this pull request you will find a set of new features.

From old issues:

New Features :

Here is a screenshot of the UI with new parameters:
image

Feel free to comment and test my work.

jgod and others added 22 commits November 21, 2014 03:36
…ing-with-bang' into integration

# Conflicts:
#	Export Layers To Files (Fast).jsx
# Conflicts:
#	Export Layers To Files (Fast)-dialog.json
#	Export Layers To Files (Fast).jsx
# Conflicts:
#	Export Layers To Files (Fast).jsx
# Conflicts:
#	Export Layers To Files (Fast)-dialog.json
#	Export Layers To Files (Fast).jsx
@finscn
Copy link
Contributor

finscn commented Sep 9, 2018

WOW , great job.

But I found there are some mistakes of my fork in new PS CC.
So I updated some codes, please look this:

finscn@46fb6aa

@Willena
Copy link
Contributor Author

Willena commented Sep 9, 2018

I started by merging your code (using your master branch).
So, your latest fixes should be there.
Maybe you can check that everything you added is still working ? (I already did a quick test before pushing and it was working)

@antipalindrome
Copy link
Owner

@Willena This all sounds very great! I'm very glad you were able to address a lot of these issues. I wish I had had more time to do some of this myself as well.

I will try to test these features, and make sure performance for the base-case is still as fast as it was before and if everything looks good I will merge it into master. :)

@antipalindrome
Copy link
Owner

antipalindrome commented Sep 11, 2018

@Willena The new features see to work really great for the most part.

I did find one issue, in Photoshop CC 2018 when I use "Top groups as layers" I get the error The command "Merge Layers" is currently unavailable

Further, even with default settings (not using any new features) when I run this script compared to the current live version, the script take on average 1.6 seconds longer (or ~17%) on export (with my test file, which is relatively small). So I'm wondering if it runs a lot of extra logic now that it might not need to?

Other than that though this works wonderfully.

If you could review the error, and see if there's anything to be done about the performance slow down that'd be great. Thanks!

@finscn
Copy link
Contributor

finscn commented Sep 12, 2018

the feature "Top groups as layers" is created by me. I'll try to fix this bug.

But , I can't reproduce it in my Photoshop CC 2018 ver 19.1.4.
Could you give the PDS file you used in test ?

@finscn
Copy link
Contributor

finscn commented Sep 12, 2018

@Willena, I've fixed the bug. (the reason is we can't merge a empty group)

I've committed the code to https://github.com/finscn/Photoshop-Export-Layers-to-Files-Fast

@Willena , could you merge it to this PR ?

if the group is empty , don't call merge()
@Willena
Copy link
Contributor Author

Willena commented Sep 12, 2018

@finscn I've merged your commit.

@hsw107 Can you do some new testing with the fix ? I'll try to find what causes the script to be a little longer, and make some changes if necessary.

@antipalindrome
Copy link
Owner

Thanks @finscn. I will retest it soon!

@antipalindrome antipalindrome merged commit 6a6f139 into antipalindrome:master Sep 17, 2018
@Willena
Copy link
Contributor Author

Willena commented Sep 18, 2018

Thanks for merging my PR.
I think It may be useful to closes all issues related to this PR ( #108 #96 #42 #79 #70 #69 #21 #64 )

@gloofer
Copy link

gloofer commented Sep 18, 2018

Excellent! Just excellent! Good Job! Definitely one of the best plug-ins for PS.

@antipalindrome
Copy link
Owner

@Willena All done :) Thanks again!

@Petermyname
Copy link

Thanks so much for this changes.

@Willena
Copy link
Contributor Author

Willena commented Sep 19, 2018

I think #56 can also be closed. If I understand it correctly, trimming wider than the layer can be done with the padding functionality (a padding value of 1px or 2px). Am I right ?

@antipalindrome
Copy link
Owner

@Willena I did miss that, thanks! And yeah, I cleaned up all the issues so they should be easier to read through.

@finscn
Copy link
Contributor

finscn commented Sep 21, 2018

@Willena I've updated my code for cosmetic . Could you create a new PR?
I'm not good at git

@antipalindrome
Copy link
Owner

antipalindrome pushed a commit that referenced this pull request Feb 8, 2020
Multiple Features - Implements old requests/issues
antipalindrome pushed a commit that referenced this pull request Feb 8, 2020
Multiple Features - Implements old requests/issues
antipalindrome pushed a commit that referenced this pull request Feb 8, 2020
Multiple Features - Implements old requests/issues
antipalindrome pushed a commit that referenced this pull request Jan 4, 2022
Multiple Features - Implements old requests/issues
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.

None yet

6 participants