Age | Commit message (Collapse) | Author |
|
|
|
This will be replaced later, so it's not logic allocate now.
|
|
Now, the name of the operator is used.
|
|
This allows to have annotation panels and grease pencil object panel at the same time.
Differential Revision: https://developer.blender.org/D6467
|
|
When the stroke weights array was NULL, the function crash. Just check NULL value.
|
|
layers
The default parameter was saved and the next time the operator was used, if it was used all layers, the default value changed.
|
|
When use `Shift+D` the original stroke was unselected only at stroke level, but not at point level and the edit line kept selected.
|
|
Add new icons and panels Grease Pencil Dopesheet to manage layers without having the properties panel visible.
Also, the icons are in the same order in Dopesheet, Layers and Material list to keep consistency.
As the number of columns for icons is limited to 3 and we also need use a factor, I have impleted the change using slider area. Also, the slider option is enabled by default for 2D Template.
See T72026 for more info.
Reviewed By: mendio, pepeland, billreynish
Differential Revision: https://developer.blender.org/D6328
|
|
When the final stroke was smaller than original stroke, the weights array must be resized, but by error the function used the original stroke pointer instead of the new stroke pointer and this corrupted the pointers.
|
|
list is empty
The problem was the draw function tried to use the material and gpsettings and both were NULL.
Now, the default material is used.
|
|
|
|
|
|
|
|
|
|
|
|
There were two problems:
1) When the buffer was reallocate, the pointer was corrupted.
2) Short variables were too small to hold long lines.
|
|
|
|
This parameter must be hidden.
|
|
|
|
|
|
|
|
|
|
Instead to do it for each action, do it only in init.
|
|
|
|
Thanks to @charlie for his help fixing this bug.
|
|
T70927
Maniphest Tasks: T70927
Differential Revision: https://developer.blender.org/D6097
|
|
The problem was the new object was created byt the tag for update objects in depsgraph was not tagged.
|
|
This patch replaces D5787. Now instead to replace the startup.blend file, all the changes are done in versioning and moved to shared module to be reused by Brush reset.
Reviewers: brecht, mendio
Reviewed By: brecht
Subscribers: pepeland, mendio
Differential Revision: https://developer.blender.org/D5913
|
|
As always uses 4 points, don't need to do calculations for that and also it was wrong with 3 points.
|
|
When using the samples, the interpolated points get abrupt steps because the system cannot receive all events in a short period of time.
This is more noticeable when the samples are set to 10 and the pen is moved very fast. The problem with post-processing smooth is that is applied to all stroke and this removes details.
The smart smooth is automatic and detect only the segments in the stroke where the system was unable to capture all movements and apply a smooth algorithm.
|
|
Ref D5197
|
|
There was a mix of old and new constants. Now have one list of WM_CURSOR_*
cursor types, using GHOST standard cursors when available and otherwise falling
back to our custom cursors.
Ref D5197
|
|
When move very fast the pen using a tablet, the end of the strokes was very ugly if the sampling was enabled. The reason for that is the last point and the previous one was interpolated in distance, but not in pressure and strength.
This commit uses several processes to get better endings:
a) If the pressure at the end of the stroke is very low, this part of the stroke is removed. This is a common issue with some tablet that send events with very low pressure when the pen is raised from drawing surface.
b) The interpolated points created by sampling are interpolated in strength and pressure to get a smooth transition.
c) Active smooth also uses the strength. Before only pressure was used.
|
|
|
|
|
|
This was a mistake when fixed the problem with multiframe sculpt. I missed change the index and use active point in the check.
|
|
If there aren't modifiers or they are not transform type, the rotation is not needed.
|
|
Instead to use the 3D cursor as reference point (legacy code), it would be better use the object location.
Tested by: @pepeland
|
|
As multiframe does not use modifiers, this is not required and only gets weird results.
Related to T70116
|
|
Related to T70116
|
|
Related to T70116
|
|
The evaluated frame is only available when the frame is the active one.
Related to T70116
|
|
The selection was not working because the evaluated frame was only working for active frame, so when the evaluation was changed to use eval data instead of original data, the data was not available and the loop did not use it.
Related to T70116
|
|
Differential Revision: https://developer.blender.org/D5716
|
|
The BKE function already does this checking.
|
|
As the layer and frame memory was duplicated, the pointers were connected to old data. This was solved when saved the file, but this was a bug. Also this required a duplication and clean all listbase items.
Now, instead to duplicate and clear memory for layers and frames, just create a new layer and frame. This solution fix the problem, it's faster and also keeps the code cleaner.
|
|
When the grab brush was used in an empty frame, a new frame was created, but as the depsgraph was no tagged, the evaluated data was wrong and the Grab hash failed.
|
|
Edit Mode
|
|
|
|
|