bb2a943350 Fix crash when the small color selector has zero size
4f47eca44e Cleanup some forgotten lines
05d5915a5a Remove pure virtual from KisColorSelectorComponent::setColor()
98f0a51d62 Add GPL2+ license to new files
4181901f1c PNG converter doesn't need half.h anymore
0347350087 Enable HDR mode in the autobuild script
- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Tue, Feb 19
The original patch was reversed. This is the correct one.
Mon, Feb 18
The patch looks and work okay here :)
Sun, Feb 17
Thu, Feb 14
Mon, Feb 11
Fri, Feb 8
In D18790#407680, @ashwind wrote:actually the setdirty call wasnt working outside the lambda command. So i was a bit confused about calling it a from a higher level.
I'm sorry, but there are a few bugs in the patch:
The updated patch fixes the pixel grid problem. Though it breaks checkers rendering :(
I have pushed the fix with a minor change: setDirty() should always be called after the change has been done, not before that. Otherwise the patch was correct. Looks like I dropped the update part when refactoring this code to the strokes framework :)
Thu, Feb 7
The problem is fixed in d1cbe53f86d5a717321db9f4c61459af4e177722
The problem is this commit 5f02b6cf33b5bb87889d40e2bad4115f4575e357
Just to make it clear, when the unmultiplication code was initially written, there were no these artifacts on the transparent background. It is some regression in another place.
- EXR images are not supposed to be viewed on transparent background. They are "light source"
- With this patch we load it incorrectly anyways. See blue artifacts on the flame.
The patch restores the bug that this "unmultiplication" was fixing. To test the issue, CandleGlass.exr image should be viewed on black background.
The setDirty() call should happen somewhere at the higher level. I guess there was some code for it. I'll check later today.
Mon, Feb 4
Fri, Feb 1
I tried to fix the wrongly enabled actions in the arrange docker (when no shapes are selected). The bug appeared because of this patch:
Hm... the idea in interesting, though I still not sure if it is obvious to the painters that -/= and Ctrl++/- shortcuts work differently.
Congratulations! Your patch seem to work perfectly fine now! Please push :)
Here is a daft patch for activating Color Picker tool with a shortcut. You can integrate it and get rid of that global option. Then I'll finally test the feature for regressions and we will commit that :)
1 | diff --git a/libs/ui/input/kis_tool_invocation_action.cpp b/libs/ui/input/kis_tool_invocation_action.cpp |
---|---|
2 | index a7b501f..792c5d4 100644 |
3 | --- a/libs/ui/input/kis_tool_invocation_action.cpp |
4 | +++ b/libs/ui/input/kis_tool_invocation_action.cpp |
5 | @@ -37,12 +37,14 @@ class KisToolInvocationAction::Private |
6 | public: |
7 | Private() |
8 | : active(false), |
9 | - lineToolActivated(false) |
10 | + lineToolActivated(false), |
11 | + colorPickerToolActivated(false) |
12 | { |
13 | } |
14 | |
15 | bool active; |
16 | bool lineToolActivated; |
17 | + bool colorPickerToolActivated; |
18 | }; |
19 | |
20 | KisToolInvocationAction::KisToolInvocationAction() |
21 | @@ -57,6 +59,7 @@ KisToolInvocationAction::KisToolInvocationAction() |
22 | indexes.insert(i18n("Confirm"), ConfirmShortcut); |
23 | indexes.insert(i18n("Cancel"), CancelShortcut); |
24 | indexes.insert(i18n("Activate Line Tool"), LineToolShortcut); |
25 | + indexes.insert(i18n("Activate Color Picker Tool"), ColorPickerToolShortcut); |
26 | setShortcutIndexes(indexes); |
27 | } |
28 | |
29 | @@ -73,6 +76,9 @@ void KisToolInvocationAction::activate(int shortcut) |
30 | if (shortcut == LineToolShortcut) { |
31 | KoToolManager::instance()->switchToolTemporaryRequested("KritaShape/KisToolLine"); |
32 | d->lineToolActivated = true; |
33 | + } else if (shortcut == ColorPickerToolShortcut) { |
34 | + KoToolManager::instance()->switchToolTemporaryRequested("KritaSelected/KisToolColorPicker"); |
35 | + d->colorPickerToolActivated = true; |
36 | } |
37 | |
38 | inputManager()->toolProxy()->activateToolAction(KisTool::Primary); |
39 | @@ -88,6 +94,9 @@ void KisToolInvocationAction::deactivate(int shortcut) |
40 | if (shortcut == LineToolShortcut && d->lineToolActivated) { |
41 | d->lineToolActivated = false; |
42 | KoToolManager::instance()->switchBackRequested(); |
43 | + } else if (shortcut == ColorPickerToolShortcut && d->colorPickerToolActivated) { |
44 | + d->colorPickerToolActivated = false; |
45 | + KoToolManager::instance()->switchBackRequested(); |
46 | } |
47 | } |
48 | |
49 | @@ -103,7 +112,10 @@ bool KisToolInvocationAction::canIgnoreModifiers() const |
50 | |
51 | void KisToolInvocationAction::begin(int shortcut, QEvent *event) |
52 | { |
53 | - if (shortcut == ActivateShortcut || shortcut == LineToolShortcut) { |
54 | + if (shortcut == ActivateShortcut || |
55 | + shortcut == LineToolShortcut || |
56 | + shortcut == ColorPickerToolShortcut) { |
57 | + |
58 | d->active = |
59 | inputManager()->toolProxy()->forwardEvent( |
60 | KisToolProxy::BEGIN, KisTool::Primary, event, event); |
61 | diff --git a/libs/ui/input/kis_tool_invocation_action.h b/libs/ui/input/kis_tool_invocation_action.h |
62 | index d23c9bc..f6f5a08 100644 |
63 | --- a/libs/ui/input/kis_tool_invocation_action.h |
64 | +++ b/libs/ui/input/kis_tool_invocation_action.h |
65 | @@ -34,7 +34,8 @@ public: |
66 | ActivateShortcut, |
67 | ConfirmShortcut, |
68 | CancelShortcut, |
69 | - LineToolShortcut |
70 | + LineToolShortcut, |
71 | + ColorPickerToolShortcut |
72 | }; |
73 | explicit KisToolInvocationAction(); |
74 | ~KisToolInvocationAction() override; |
In D16833#389519, @emmetoneill wrote:
I've just pushed William's patch into master. The new mode can be configured globally in Krita settings (it is on by default).
Thu, Jan 31
That is not a problem either. As Deevad has shown in his video, the old effect
is still achievable.
I have checked the maths of the new algorithm. It looks logical and should work fine. I have also added a few comments to the code to make it easier to decrypt later. See patch snippet below.
Tue, Jan 29
Mon, Jan 28
I need to check the maths of it
I need to test this :)
Sat, Jan 26
In D18412#399923, @woltherav wrote:Hm... I am not sure if it is wise to have this patch if it only changes the wraparound view but not the wraparound strokes.
- BUG: Pixel Grid doesn't care about custom WA mode settings. It is painted in both directions whatever the settings are
Wed, Jan 23
Except double redo() of the command the patch looks kind of okay. I don't feel good about making the dialog non-modal, because it allows the user to open it multiple times. But the layer properties dialog is already affected by this bug, so it will not be too big regression.
Outch, I didn't intent to accept the patch. It needs fixes :(
The dialog after the patch seem to work. The only trouble I saw is some debugging output, it is seem to have been present before the patch. Can be removed though :)
The main thing that should be checked in such cases is the presence of deadlocks when trying to do some actions, when the dialog is open. E.g. opening this dialog again, or starting a filter, or changing the current layer or removing the current layer.
I have tested your patch. It looks like the rendering problem has gone, indeed. Though there are a few other smaller problems :(
Ah, if it is going to the the only ZIP provider, then it should be fine. Though we should check its Unicode support. I remember we had some troubles with people saving .kra images with Russian image names in it.
There is a problem inherited from the previous patch: copy-constructor and assignment operator do not copy the values for the precision. Therefore undo/redo will not work properly. Please add copying to ToolTransformArgs::init() function.
Jan 21 2019
Here is the full tablet log for exactly two strokes. The first one with the normal "left" button, the next one with the overridden "Mouse 12 Button". It behaved exactly as in the video above. It dropped the stroke right after the start. From the log it looks like "Mouse Move" events do not have proper "buttons" field set. Therefore it recognizes teh button as being released.
There is some problem with rendering when using this patch:
I am okay with this very patch for adding the actions. They look okay and work fine.
The patch looks okay now. Though it lacks a necessary part: saving of the options to the config file. If you promise to finish the feature later, we can push it as it is (with the two stylistic changes I noted inline) :)
Jan 20 2019
I still have a crash when trying to use your option. It looks like the config value gets uninitialized. I don't know how it happens, but it is not initialized :(
Here is the output:
When setting precision of the final result to '2', Krita crashes because of trying to allocate 0xffffffffffffffff bytes. I guess this precision level is not supported by the grid algorithm.
In T9256#173995, @troyjamessobotka wrote:The scale is there to convert the conventional scaling of values to nits. That is, 0.18 would become 18 in ST.2084.
Jan 19 2019
A patch for @troyjamessobotka's OCIO config for conform Windows convention:
Jan 14 2019
In D17807#385686, @hellozee wrote:Is there any other place where pixelPrecision value could be hardcoded? Cause if any value other than 16 is passed, it causes an assert.