Blender Git Commits

GP Add Primitive: Don't make a copy of the new stroke. Just copy it over!

Cleanup: Streamline logic here - just check once, not twice

GP Add Primitive Ops: Remove the "avoid fast double clicks" event hack

This was making the operator unnecessarily clumsy to use, especially
for anyone used to the way that drawing tools everywhere else work.

Old workflow (with the hack):
1) Click Rectangle/Circle to start the operator
2) Click (Press & Release) in the 3D view to start drawing the shape
3) Click again (Press & Release) to stop drawing
(4) Click again, realising that you forgot to release the mouse in step 2
(provided you realised why it "wasn't working")

New workflow (more in line with standard workflows):
1) Click Rectangle/Circle to start the operator
2) Press and drag in the 3D view to start drawing the shape
3) Release to stop drawing

GP Add Primitives: Allow operator to work in Edit Mode too

It didn't really make sense why this was restricted to Draw mode only.
In particular, there are 2 good arguments for allowing it in editmode too:
1) In many ways, it's more similar to a Copy/Paste or Duplicate
operation in that it just dumps a bunch of stroke geometry
into the scene without you having to draw anything. We allow those,
so why not this.

2) Users are used to being able to go into Edit Mode and add a primitive
shape into the datablock to begin editing it. For consistency with
other modes, and also to prevent the "Creation" tab looking stupid
and redundant in EditMode, let's enable this.

Note:
* Sculpt/Weight Paint modes do not have this enabled for now. In those
modes, users are less likely to be concerned with adding geometry,
and more with modifying the existing geometry.

Cleanup: Use M_PI instead of converting from degrees to radians

Note: While the center/radius calcs may benefit from using the
BLI_math funcs, the current implementation is more direct.

GP Add Primitives: De-duplicate code

Most of the logic in gp_primitive_rectangle() and gp_primitive_circle()
was duplicated, and responsible for performing space-conversion stuff
on the 2d/screen-space coordinates.

This commit properly separates out these concerns, de-duplicating
code, and fixing a few other minor issues at the same time
(e.g. superfluous copies, and "edges" not getting initialised properly)

GP Add Primitives: Removed the requirement for an active layer

Previously, the add primitive operator (rectangle/circle) required
there to be an active layer present before it would allow you to draw
a new primitive. However, this reduced the usefulness of this tool
(especially if you intended to use these tools to roughly block out
the requried shapes first in a new/blank GP object).

There was no real reason for this limitation, so no, the operator will
create a new layer itself if necessary, just like the Draw operator
has done since day one!

Cleanup: Remove redundant functions

These were just adding extra layers of abtraction/complexity
without serving any actual purpose.

Rename "Chessboard" to "Checker Board"

Support multiframe for change color operator

Fix Undo problem with color picker

Redesign off GP icons

Designed by Matias Mendiola

Thanks Pablo Vazquez for the feedback

Merge branch 'blender2.8' into greasepencil-object

Enable drawing start with Shift + Mouse

This is necessary for lazy mode start

Change Pattern drawing to use standard functions

Now the drawing is using the standard functions instead of custom drawing.

Thanks Pablo Vazquez for the trick

Disable brush temp override for lazy mouse

Use Shift key to toggle Lazy Mouse

Now while drawing you can enable or disable the lazy mouse with the shift key.

If the brush has lazy enabled, shift key disable it. If the brush has lazy disabled, shift key enable it.

Replace Polygon Keymap to Ctrl+Shift

This keymap has been changed in order to free the shift key for Lazy Mouse

Remove Select keymaps in Drawing mode

These keymaps were enabled by error in this mode

Fix cursor error drawing polygons

The cursor had a wrong line due lazy mouse cursor mode that must be hidden in poly mode.