Blender Git Loki

Fix logical error in BLI_hash_pointer_to_color

'hash_b' was always zero as it's range was bit-shifted away.

Merge branch 'blender-v2.83-release'

Fix normal smoothing for light positioning gizmo

Merge branch 'blender-v2.83-release'

Fix T75794: Light point gizmo translate jumps

Holding Ctrl while dragging the light point gizmo would use
uninitialized stack memory if the normal had not been
initialized by a surface.

Now holding Ctrl can be used to drag,
even when there is no surface to orient to.

Merge branch 'blender-v2.83-release'

Fix T75796: Misaligned quit dialog keymap

Merge branch 'blender-v2.83-release'

GPencil: Add always empty frame when add new layer

The dopesheet needs to have a frame to display the channel, so an empty frame is created in the current frame.

See T66505 for details of why an empty channel cannot be displayed.

UI: support A-Z accelerator keys for pie menus

Converting menus to PIE's was removing convenient key accelerators.

Fix toggling quad-view loosing 3D view clipping

UV: minor adjustments to opacity adjustment

- Allow 0.0..1.0 range, as even at 0.0 the selection is still visible.
- Correct versioning code, not to overwrite the value for new files.

GNUmakefile: remove style checking targets

This has been removed since clang-format now enforces code-style.

Cleanup: comments for ui_draw_menu_item & correct argument name

Cleanup: use colon after doxygen parameters, spelling

UI: Fix bad flow layout

The layout of the new sequencer disk cache were not handled well with
large preference windows.

GPencil: Add always empty frame when add new layer

The dopesheet needs to have a frame to display the channel, so an empty frame is created in the current frame.

See T66505 for details of why an empty channel cannot be displayed.

UI: Remove old hacks for dynamic scrollbar hiding

Although we still dynamically hide scrollbars, they don't change the
region size anymore. They are simply drawn on top of the region content.
Because of this, some hacks introduced by fa28e50ac2a7 are no longer
necessary.
Without these hacks, the scrollbar visibility is evaluated much more
often (cheap operation) which should be more reliable and possibly solve
some glitches.

Also replaces integers passed as booleans.

Fixes T75782.

Merge branch 'blender-v2.83-release'

Fix T75675: Unlinking [with setting users to zero] not clearing
LIB_TAG_EXTRAUSER_SET flag

For example in the Image Editor, an assert would be triggered after
unlinking an image [with setting users to zero] and then setting the
image for the Image Editor again.

Whenever we set an Image for Image Editor, the Image ID is flagged
LIB_TAG_EXTRAUSER_SET, when we unlink [with setting users to zero] this
flag was not cleared.

quote @mont29: "a proper fix would be to move this to modern code, and
actually delete the ID..." but that is for later.

Maniphest Tasks: T75675

Differential Revision: https://developer.blender.org/D7452