search

adobe-photoshop / spaces-design

Adobe Photoshop Design Space
http://adobe-photoshop.github.io/
Other
853 stars 74 forks source link

Automatically turn off "Add copy to to copied layers and groups" #3543

Closed placegraphichere closed 8 years ago

placegraphichere commented 8 years ago

We should not add "copy" to all duplicated layers. Its annoying. Either maintain the setting a user has or just turn it off by default in Ds.

Received through user feedback

screen shot 2016-01-14 at 7 55 40 pm

Lemme know if this should go in JIRA

iwehrman commented 8 years ago

This doesn't have to happen in the current sprint @volfied but if you can find a simple flag to pass to our action descriptor, this would be nice.

baaygun commented 8 years ago

I can fix this in a simple way by exposing that option to the UAppElement property, since it's globally available in Photoshop. Then we can set it ourselves on startup to the value we like.

baaygun commented 8 years ago

I will work with Jesper on making sure this property is bounced back to existing value in Classic.

iwehrman commented 8 years ago

FBNC to @chadrolfs.

chadrolfs commented 8 years ago

Personally, I didn't see a problem with the way it was before this change - DS would honor whatever the standard PS pref was so if you didn't want 'copy' added, just turn the pref off, enter DS and work away. The old way, you had the option either way - this way, we're forcing the user to user to work with it off.

Other than forcing it off by default with no other option to turn it back on, the only other workflow problem with this is with Export. You can have 5 artboards or layers with the same titled/name (e.g. Artboard 1), but only 1 will get exported.

baaygun commented 8 years ago

@chadrolfs that is an important point. @mcilroyc is this going to break export, as in is export reliant on layers having different names?

iwehrman commented 8 years ago

Also, @placegraphichere, can you jump in here and argue for not exposing this pref (via standard PS) in Design Space?

baaygun commented 8 years ago

By the way, I think that export code should gracefully handle multiple layers with the same name, maybe dropping down to parent names to show shortest unique path in the exported file name. After all, this option of appending "copy" is something the user can achieve themselves by renaming layers, as well.

mcilroyc commented 8 years ago

I agree that export should be better about multiple layers with the same name, so it is irrelevant to this conversation.

chadrolfs commented 8 years ago

I think that issue can be tracked via #2274