- User Since
- Oct 31 2013, 3:14 PM (457 w, 5 d)
Sun, Jul 31
Tue, Jul 26
Agree, until the opacity feature is not implemented for curves it is better to hide it for Grease Pencil also. Maybe you can open a new task for this @Antonio Vazquez (antoniov)
Jul 7 2022
Jun 29 2022
Overall the patch seems to be working really nice.
Jun 25 2022
Jun 21 2022
Jun 20 2022
Thanks @Amelie Fondevilla (afonde) , great work!
From artist perspective everything work fine, the two missing thing (datablock summaries and gray-out Fcurve operators in menus) could be addressed in another patch.
Jun 10 2022
Right, the Set Color Attribute is not working as expected
Jun 4 2022
Great progress @Amelie Fondevilla (afonde) ! All work like a charm.
As soon as you finish with the missing thing in your list the patch could be approved.
Jun 1 2022
May 23 2022
May 20 2022
Thanks @Amelie Fondevilla (afonde) for the proposed fix for a so long awaited feature, it look really promising!
I know is a WIP but I leave my first notes after testing the patch:
May 19 2022
As you said @Antonio Vazquez (antoniov) the tool is intended to use only on opened strokes, can we just ignore closed strokes in the algorithm to avoid undesired results?
May 18 2022
Since this feature is so useful to the storyboard workflow I've added together with https://developer.blender.org/D11890 to the storyboard main task https://developer.blender.org/T68321
It seems so close to being fully reviewed!
May 17 2022
May 13 2022
May 10 2022
May 6 2022
Apr 28 2022
Apr 26 2022
Apr 21 2022
Totally agree with @Pablo Vazquez (pablovazquez) , If the Grease Pencil filter button is not working as expected we should consider this a bug
Apr 20 2022
Apr 12 2022
Apr 7 2022
All works as expected
Mar 30 2022
Add more discoveratibility to this brush settings is totally right, but we have to follow the same way Blender does with Sculpt brushes for meshes: using a Brush popover menu on the header.
Mar 29 2022
Mar 26 2022
Mar 25 2022
Some notes after testing:
Mar 23 2022
The pie menu item addition is logic, the patch is Ok for me
@Matias Mendiola (mendio) I think the thickness slider in deform mode is quite nice, since the usual usecase, that I am optimizing for would otherwise always require a thickness modifier before with thickness = 0. Since I am changing the thickness in the modifier anyway and since I have the slider already from the other modes, I may as well just include it and make the modifier stack more compact for the most common usecase.
The most common usecase for the deform mode that I am talking about, is using it as an alternative to the thickness from curvature modifier stack, mostly when doing handwriting.
The modifier works much better now.
Mar 22 2022
Thickness, Strength and Material Index are only relevant for Segment and Fill modes, they should appear only in those modes in the UI
Mar 21 2022
Thickness, strength and material index addition add a lot to the usability of the modifier, thanks
I also like the use of options instead the checkbox.
After some testing here are my notes:
After some testing here are my notes:
Mar 3 2022
Mar 1 2022
The goal of this patch is try to improve the actual storyboard workflow with a small UI/UX change.
Feb 25 2022
All my undo tests with the Draw tool worked lot faster. Great improvement
Feb 14 2022
For better discoverability I suggest changing the UI name to Dilate/Contract
Feb 13 2022
it's hard for me to get nice and smooths lines results using the modifier without the use of the Simplify Modifier first, I wonder if could be possible to improve the code to avoid that
Feb 8 2022
Feb 2 2022
Nov 26 2021
Nov 4 2021
Nov 1 2021
The operator works better now.
Flatten layers is a destructive operator, so it's up to the user to set up Modifiers accordingly and IMO all layers should be flattened regardless is locked or not (All other softwares that have similar operators behave this way).
It seems that the layer flattening result are different depending on the current active layer. The result should always be the same regardless the active layer
Sep 30 2021
Sep 25 2021
Sep 24 2021
Sep 23 2021
Sep 22 2021
Sep 20 2021
Sep 6 2021
Note: Actually original design of some of the icons was by @YimingWu (NicksBest) , I just tweaked the final implementation
Aug 26 2021
Please don't forget that Line Art is a Grease Pencil Modifier, and as we discussed before, the main problem with the In Front aproach by default is that it doesn't work well with other Grease Pencil object in the scene.
For 2D Artist is key to have a nice integration between a 3D NPR background using line art and their 2D animations using standard Grease Pencil object. The Stroke Offset property allow that integration.
Aug 23 2021
Aug 3 2021
Jul 24 2021
Jul 22 2021
Nice Antonio, functionality is much better now.
This option would be useful for primitives. With the patch it is only possible to access the new brush property using the Draw tool, maybe this property should be outside any particular brush menu
Jul 14 2021
Jul 10 2021
Some initial notes:
Jul 1 2021
The right default distance is tricky because it depends on many factors (Camera distance, Size of the strokes, distance between objects/strokes, etc) and also on personal taste.
For my tests I used the standard camera and 2D-Monkey: 0-50 works well for separation of 3 meters between the front and back strokes and 0-20 for a separation in the range of 1m
Jun 29 2021
exactly, this could be reviewed/updated in the new GPencil-Eevee development
This is a workaround that helps to display the Gpencil object with more accurate colors in dark environments without affecting the lighting on meshes. Remember that Grease Pencil is designed for 2D art.
Jun 28 2021
Lock Active is On by default not only for Grease Pencil but Meshes, so a change in the default value should have to be consistent in both.
This task could be closed and another one created with a more general proposal if wanted.
Jun 25 2021
Jun 23 2021
Separation in two panel was by design to control the annotations layer visibility regardless the user is using or not the Annotate tool.
If layer visibility control are in the tool setting panel the user have to change to the Annotate tool everytime they want to change notes visivility.
Jun 21 2021
Jun 9 2021
@Jeroen Bakker (jbakker) This commit is good for 2.93 LTS