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

Border style topDivider added to documentation. #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SetTrend
Copy link
Contributor

The Adobe Photoshop script "Image Processor.jsx" at
C:\Program Files\Adobe\Adobe Photoshop 2022\Presets\Scripts
comes with a border style value of topDivider for a number
of panels.

This value draws a gray line at the top of a panel:

Image Processor Dialog (original)

The topDivider value is currently undocumented. This pull
request adds the corresponding documentation.

The Adobe Photoshop script "Image Processor.jsx" at
`C:\Program Files\Adobe\Adobe Photoshop 2022\Presets\Scripts`
comes with a border style value of `topDivider` for a number
of panels. This value draws a gray line at the top of the panel.

The `topDivider` value is currently undocumented. This pull
request adds the corresponding documentation.
@zlovatt
Copy link
Member

zlovatt commented Mar 14, 2022

Other considerations and notes that should be added: does this work in every hostapp? How long has this been part of Extendscript?

If it's only specific to Photoshop 22+, then a warning should be added noting this.

@SetTrend
Copy link
Contributor Author

Hi @zlovatt!

As I replied in my other PR to your request: This is something only an Adobe employee can answer. I'm just a user trying to help. I don't have any insights of when or where this feature was added by Adobe.

@zlovatt
Copy link
Member

zlovatt commented Jun 17, 2022

@SetTrend Also verifiable by going and checking various hostapps and versions of PS yourself! It's all anyone else is doing, and would go a long way to ensuring the documentation is accurate.

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.

2 participants