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

Add Opacity slider to Appearance panel #1078

Closed
neilweinstock opened this issue Jan 25, 2022 · 4 comments · Fixed by #1197
Closed

Add Opacity slider to Appearance panel #1078

neilweinstock opened this issue Jan 25, 2022 · 4 comments · Fixed by #1197
Assignees

Comments

@neilweinstock
Copy link
Contributor

neilweinstock commented Jan 25, 2022

Unstable release adds support for alpha channel in the 3D renders. Right now the only way to control it is inside the color picker, in the RGB/HSL/blah blah blah section. This is far too hidden and too inaccessible. Propose a slider, labeled "Opacity", somewhere under the color button, either above or below the "Shine" control.

The Alpha channel controls in the color picker could still be there, it doesn't hurt to have it in both places. Obviously, editing in either place should show up in the other.

See #401, which is related but not quite the same.

@hcraigmiller
Copy link
Collaborator

hcraigmiller commented Jan 25, 2022

See comment below.

@neilweinstock
Copy link
Contributor Author

neilweinstock commented Jan 25, 2022

There's already an alpha channel setting there. It works and is fine. No need to go mucking with the color picker. Purpose here is simply to surface the same functionality in a more friendly way, e.g.:

image

I would really prefer the opacity slider to go from 0-100% (as shown above) but it may be much easier to just have it be 0-255 and therefore be exactly the same as the alpha channel value. Otherwise need to convert back and forth, which is possible but not idea.

@hcraigmiller
Copy link
Collaborator

I agree that the opacity (transparency) slider should go from 0-100% as suggested by @neilweinstock.

@neilweinstock
Copy link
Contributor Author

I may try to implement this myself at some point if no one else gets to it first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants