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

Changing layer opacity locally. #960

Open
abrasivetroop opened this issue Mar 18, 2021 · 10 comments
Open

Changing layer opacity locally. #960

abrasivetroop opened this issue Mar 18, 2021 · 10 comments
Labels
feature New features or improvements to existing ones needs discussion For ideas not fully fleshed out yet and need more work before putting into motion

Comments

@abrasivetroop
Copy link

I would like to change the opacity of a layer locally but unfortunately it is controlled by the operator. A lot of people draw a quick sketch and then reduce the opacity of that layer and start drawing on top of that layer. But this is not possible in drawpile because it would change the opacity for all users :( I was wondering if it is possible to implement this feature? It would be so good. Thanks!

P.S. - Thanks for this great app! I really love it :)

@blurymind
Copy link

blurymind commented Mar 18, 2021 via email

@notfood notfood added the feature New features or improvements to existing ones label Mar 18, 2021
@notfood
Copy link
Contributor

notfood commented Mar 18, 2021

As a workaround, you can use a big eraser with lower opacity and flat drawing mode. It'll lower the opacity of whatever you have.

@Wade821
Copy link
Member

Wade821 commented Mar 18, 2021

This is more trouble than it's worth. It'll lead to things looking different for different users and possibly lead to confusion trying to figure out how to reset it back to the operator controlled view. I personally don't think it's a good idea as it detracts from the idea of a drawing board for many people. As @notfood said, that would be a better work around to do that.

As such, I'm going to change the labeling for this unless we have a better reason to use it. Feel free to discuss further.

@Wade821 Wade821 added needs discussion For ideas not fully fleshed out yet and need more work before putting into motion invalid Not related to Drawpile or a bug report for something that's working as intended and removed feature New features or improvements to existing ones labels Mar 18, 2021
@askmeaboutlo0m
Copy link
Member

(I've taken over development on Drawpile, so I'm going through these tickets and organizing them.)

Drawpile 2.2 has local canvas backgrounds to solve a similar issue: you want a different background color, but only operators can change it. Wanting to change the opacity of a layer seems like a very similar case to me, I think it's just a matter of figuring out a good user interface for it that isn't confusing. Allowing arbitrary parameters to be synchronized would be technically possible, but would be overkill.

Then again, with canvas backgrounds there's no workaround, with coloring sketches there totally is by just lightening up your lines. So I'll leave this open for now, just relabeling it.

@askmeaboutlo0m askmeaboutlo0m added feature New features or improvements to existing ones and removed invalid Not related to Drawpile or a bug report for something that's working as intended labels Mar 23, 2023
@abrasivetroop
Copy link
Author

(I've taken over development on Drawpile, so I'm going through these tickets and organizing them.)

Drawpile 2.2 has local canvas backgrounds to solve a similar issue: you want a different background color, but only operators can change it. Wanting to change the opacity of a layer seems like a very similar case to me, I think it's just a matter of figuring out a good user interface for it that isn't confusing. Allowing arbitrary parameters to be synchronized would be technically possible, but would be overkill.

Then again, with canvas backgrounds there's no workaround, with coloring sketches there totally is by just lightening up your lines. So I'll leave this open for now, just relabeling it.

Hey! Thanks a lot. First responses to my request was quite obnoxious and they immediately labeled it as Invalid without even giving it a thought. This is could be a very useful feautre if implemented right.

@Wade821
Copy link
Member

Wade821 commented Mar 24, 2023

(I've taken over development on Drawpile, so I'm going through these tickets and organizing them.)
Drawpile 2.2 has local canvas backgrounds to solve a similar issue: you want a different background color, but only operators can change it. Wanting to change the opacity of a layer seems like a very similar case to me, I think it's just a matter of figuring out a good user interface for it that isn't confusing. Allowing arbitrary parameters to be synchronized would be technically possible, but would be overkill.
Then again, with canvas backgrounds there's no workaround, with coloring sketches there totally is by just lightening up your lines. So I'll leave this open for now, just relabeling it.

Hey! Thanks a lot. First responses to my request was quite obnoxious and they immediately labeled it as Invalid without even giving it a thought. This is could be a very useful feautre if implemented right.

The problem was that Calle was barely around to do development work. With a timeframe potentially spanning anywhere from a couple of months to years, requests for changes that could potentially be obnoxious to implement wasn't practical and thus labeled as invalid. Now that it has changed who is the main dev, we have more options.

And while it's possible to do... what happens if a board operator changes the opacity after you've set it locally? What happens if the board operator changes the blend mode? How many settings should be made locally adjustable if the board operator can change them, and which mode should take priority? How does one display the current settings that the operator has set vs what you have set? I still don't think this is a very practical request in the long run and just leads to a more confusing, complicated interface. That's why I said before, feel free to discuss and offer ideas on how to implement it in a way that makes sense and keeps things simple.

@askmeaboutlo0m
Copy link
Member

Yeah, I agree with the issues too, I'm not sure how to solve them in a sensible way. I'll leave this standing as-is for now.

@leandro2222
Copy link

leandro2222 commented Aug 13, 2023

I totally agree. People need to change the opacity multiple times while drawing to see/unsee the rough sketch lines so imo it should be local (each user controls its layers opacity) no doubt.

I made this same request in this old post if someone is interested: #445

@askmeaboutlo0m
Copy link
Member

Well, they don't need to change the opacity to lighten the lines of a sketch, they can also select the area they want to lighten/darken, pick a lighter color and Edit > Recolor Selection (Ctrl+Shift+Comma). The problem is if they want to do it but they can't because the session uses some permission structure that doesn't let them or because they're drawing with multiple people on the same layer.

The solution is tricky because of the various issues listed above. It definitely shouldn't be local-only, that would make it effectively impossible to ever sensibly do shading and lighting in Drawpile because it'd keep forgetting your opacities every time you reconnect. It must either be global-only, as it is right now, or have some kind of local override, which has unsolved problems.

@askmeaboutlo0m
Copy link
Member

I've been told about a feature in Clip Studio Paint that was described to me as "bluescaling". It lets you turn the contents of a layer light blue, allowing you to sketch over it easier.

That seems like it could be a solution: instead of letting you control opacity locally, there's a "sketch mode" you can activate on layers, which will turn down the opacity and make it blue or something by default, with the specific parameters being adjustable. I think this would solve the issues with UI complexity and such, since it's a clearly distinct mode, rather than some overloading of the existing opacity controls.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New features or improvements to existing ones needs discussion For ideas not fully fleshed out yet and need more work before putting into motion
Projects
None yet
Development

No branches or pull requests

6 participants