Blender Git Commits

UI changes for color picker

* Colors have labels, so previews don't need to be as big, GP_BOX_SIZE set to 32.
* More padding between items, they are too close to each other at the moment GP_BOX_GAP set to 24.
* Round boxes are a bit less round to match the rest of the UI 0.2f (before used 0.4f)
* Cursor should not change to color picker. We are performing an action (selecting an item), not picking UI colors per se.

Thanks Pablo Vazquez for the feedback

Merge branch 'blender2.8' into greasepencil-object

Cleanup: Define a shared poll method for edit 3D

Cleanup: Remove unnecessary flags and fix typo

New Separate operator

This operator works similar to mesh separate operator, but for grease pencil strokes.

Two options:

a) Selected Strokes
b) Active Layer

Move int <-> float 2d vector funcs from GP code to Math Lib

We now have the following two new functions:
* copy_v2fl_v2i(float r[2], const int a[2]) <-- Copies 2D int vector to float vector
* round_v2i_v2fl(int r[2], const float a[2]) <-- Copies 2D float vector to int vector, by rounding the values

Merge branch 'blender2.8' into greasepencil-object

Cleanup: Use standard mathlib functions here, instead of inlined pointer magic/etc.

More self-contained fix for previous commit

Cleanup: Various fixes

* Remove redundant operations (e.g. extra copies of data)

* Remove duplicate functions (e.g. vector copying, for non-floats)
TODO: We still have copy_v2int_v2float() and copy_v2float_v2int()
to fix (i.e. rename + move to BLI_math.h)

* Cleanup redundant struct forward defs in headers (including some duplicates)

* Hacky fix for compiler warning in gpencil_colorpick.c (about const vs non-const)
UI_FSTYLE_WIDGET is const, but UI_text_clip_middle_ex() needs non-const

Cleanup: More simple fixes

Cleanup: Style fixes

Cleanup: Move ED_gpencil_stroke_minmax() to blenkernel

The rest of the boundbox/minmax code is already there. We might as well
have the function for grabbing this info from a single stroke there too.

Cleanup: Reshuffling order of code in file

In future: Keep related code together, instead of just dumping
it "wherever" (aka at the end of the file, since it's most
convenient right now)

(In the process, I've ended up flagging a few pieces of code
for further investigation. They likely duplicate some code
elsewhere in other modules)

Cleanup: Prefix all exported functions with module prefix

Cleanup: Fix mixed use of tab/space whitespace

Cleanup: Move private/internal operator state data out of public header for GPencil module

This commit moves the defines for several modal operators
(Interpolate, Fill, and Add Primitives) out of the editors/gpencil
module's public headerfile ("ED_gpencil.h") into the module's
internal headerfile ("gpencil_intern.h").

We should not be spraying the internal implementation details of
operators in public headers. In this case, these defines weren't
even being used outside of the gpencil module - they were only
put in a header so that drawgpencil.c (which lives in the same
module) could access this data (and even then, I wonder those
callbacks need access to everything).

Note: I couldn't get rid of the ED_numinput.h include in the header
yet, as several of the operators define NumInput members inline.

Fix error in fill stroke thickness

To avoid gaps, the thickness of the internal strokes must be reduced, but the previous code only worked when the factor was different of 0.

Merge branch 'blender2.8' into greasepencil-object

Rename "lazy Mouse" to "Stabilizer"

Also changed "Radius" to "Distance"

Thanks Pablo Vazquez for the feedback.