Explain PressureIn sensor input
ClosedPublic

Authored by razvanr on May 28 2018, 10:15 PM.

Details

Summary

Added explanation for PressureIn and slightly changed the Pressure to make it more distinct and correct cause the tablet doesn't have the pressure sensor, it's actually the stylus that does that...

The PressureIn explanation is a bit lengthy but I can't think of a simpler way to do it. If any of you can summarize what I've wrote I'm open to any suggestion on how to make it more concise.

Diff Detail

Repository
R1012 Krita.org Documentation Website
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
razvanr requested review of this revision.May 28 2018, 10:15 PM
razvanr created this revision.
razvanr edited the summary of this revision. (Show Details)
razvanr updated this revision to Diff 35063.May 28 2018, 10:20 PM

fixed some typos

woltherav requested changes to this revision.May 28 2018, 10:35 PM
woltherav added a subscriber: woltherav.

I know what you're trying to say, but that analogy is very abstract.

Perhaps it is better to explain what pressure-in is trying to solve (Afaik, it was a solution to a reverse pressure curve on size. The person who made it didn't like how the reverse pressure curve on size would mean that the brush would become bigger when pulling the stylus up)?

This revision now requires changes to proceed.May 28 2018, 10:35 PM

@woltherav I find that original explanation very problematic, first of all it doesn't solve at all what it's trying to solve. Second it focuses only on size whereas PressureIn is generic, is not about the size setting. Honestly I tried to explain what PressureIn does without the analogy and it's even more confusing. But I'm willing to revise it if you have any other ideas, like I said, explaining it in terms of the original solution is pretty bad, it took me one full day to even understand PressureIn based on that explanation in the first place and it wasn't until I extensively played with the setting to understand what it actually does. And honestly I don't even think I can explain it in those terms, because I don't believe it myself. It doesn't do what the original author says it's supposed to do.

Maybe it's best if we make a diagram, what do you think?

If you make a diagram, you will need to make it for all sensors.

The problem is that this is a wall of text, and I am not going to accept something in the manual I barely can read myself. On top of that, the purpose of the wall of text is to make something clear, but instead you end it with 'It's best to experiment with it to understand how it behaves', making the whole wall of text useless.

Just remove the analogy or come up with a better one, or just sit down and explain the user that pressure in only registers pressure change when the resulting pressure is higher than the previous pressure it was registering over the stroke.

Restricted Application edited projects, added Krita: Manual; removed Krita. · View Herald TranscriptMay 29 2018, 9:52 AM
Restricted Application added a reviewer: Krita: Manual. · View Herald Transcript
rempt added a comment.May 29 2018, 9:55 AM

Yes, I think that the proposed text doesn't really clear things up.

razvanr added a comment.EditedMay 30 2018, 5:16 AM

I agree, the text is very lengthy and still more confusing than not. I'll rework it later with the critique in mind and I think best is something in the lines of what you mentioned @woltherav.

razvanr updated this revision to Diff 35230.May 31 2018, 6:54 AM

Remove ugly wall of text, simplify explanation, hope this is better.

woltherav accepted this revision.May 31 2018, 7:24 AM

I'll replace 'lessening' with 'reducing' and will push it later.

This revision is now accepted and ready to land.May 31 2018, 7:24 AM
This revision was automatically updated to reflect the committed changes.