Age | Commit message (Collapse) | Author |
|
Remove redundant code for drawing text strings that contain only ASCII.
See D12293 for much more detail.
Differential Revision: https://developer.blender.org/D12293
Reviewed by Campbell Barton
|
|
gizmo_button2d_bounds result wasn't valid when the gizmo was part
of a 3D gizmo group.
Regression in cf6d17a6aa421e0038fc1f8e60e3f1f708887c3e
|
|
Passing 4x arguments for the rectangle,
mixed in with round-box radius & color wasn't very readable.
Instead, pass a `rctf` as the first argument to UI box drawing functions.
|
|
|
|
Bounding box calculation ignored the gizmo size preference.
|
|
Position Gizmo tooltips below their bounds so they do not obscure the content.
Differential Revision: https://developer.blender.org/D9793
Reviewed by Julian Eisel
|
|
The float array is declared inside a smaller scope, pointed to, and then
used later. This simply moves the declaration outside of the if statement.
Also remove two redundant lines.
|
|
Simplification and changes to the Navigation gizmo. Better indication of negative axes, consistent use of color and size to indicate orientation, ability to be resized.
Differential Revision: https://developer.blender.org/D9744
Reviewed by Campbell Barton
|
|
Following the most widely used convention for including todo's in
the code, that is: `TODO(name):`, `FIXME(name)` ... etc.
|
|
Also increase text size from 11 to 12 since the default
font at 1.0 scale was a little fuzzy too.
Reviewed by: @pablovazquez
Ref D8781
|
|
This improves circles AntiAliasing, and line antialiasing.
This keeps the old drawing method (3d spheres) for the selection pipeline.
This was suggested by @harley on devtalk.
|
|
The gizmo one was only reproducible in debug builds.
The GLImmediate one was only affecting amdpro drivers when --debug-gpu was
enabled.
|
|
We now use GPU_blend for enabling / disabling blending and explicitly
set the blend equation.
|
|
This change is yet to be followed by a more comprehensive design
proposal including:
* How to differentiate the modes apart.
* More clear definition of tools and the rules for their components (gizmo, cursor).
* Selection as a non-tool vs drag option.
This can be revisited for 2.90 with more time. For now the UI team
agrees to revert this.
--
This reverts commit 4aa703aa1430bc53f19e2cc7182e70db1a916f13.
|
|
This new shader is able to emulate smooth wide lines drawing using a
geometry shader.
This shader needs viewportSize and lineWidth uniforms to be set.
There is multiple variants to replace the usage of wide lines for most
shaders.
This patch only fix the gizmo_types files and the navigation gizmo.
Other areas could be fixed afterward, I just limited the patch size.
Fix T57570.
Reviewed By: billreynish
Differential Revision: https://developer.blender.org/D7487
|
|
|
|
|
|
Normal UI widget and 3D navigation gizmo where clipping at high DPI.
|
|
|
|
Changes the default View3D mouse cursor to the OS-supplied arrow pointer. Subsequent cursor changes will now be set per-tool instead.
Differential Revision: https://developer.blender.org/D6485
Reviewed by Campbell Barton
|
|
Happened when used in 2D view.
|
|
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
|
|
|
|
Apply clang format as proposed in T53211.
For details on usage and instructions for migrating branches
without conflicts, see:
https://wiki.blender.org/wiki/Tools/ClangFormat
|
|
Without this they can blend in visually with the backdrop of the gizmo.
|
|
Using white with low alpha made it hard to use on a white background.
Since the axes themselves highlight to white.
|
|
While \file doesn't need an argument, it can't have another doxy
command after it.
|
|
Move \ingroup onto same line to be more compact and
make it clear the file is in the group.
|
|
BF-admins agree to remove header information that isn't useful,
to reduce noise.
- BEGIN/END license blocks
Developers should add non license comments as separate comment blocks.
No need for separator text.
- Contributors
This is often invalid, outdated or misleading
especially when splitting files.
It's more useful to git-blame to find out who has developed the code.
See P901 for script to perform these edits.
|
|
|
|
Prevents clang-format wrapping text before comments.
|
|
|
|
Show occluded axis highlight on top when pointing away.
|
|
Axis aligned views now show both small/large handles,
this makes it possible to well if the view is in front or behind.
|
|
Previously the positive axis was always brighter,
now use bright colors which face towards the view.
|
|
When view aligned, the near axis was hidden,
making it seem as if the opposite axis was at the front.
|
|
Also use struct for storing extra args.
|
|
|
|
Using low alpha meant it wasn't possible to see which was in-front.
|
|
|
|
|
|
This now matches original design more closely, see: T54723.
|
|
Much easier to read against the usually brighter axes colors.
|
|
- Only show axis backdrop when hovering.
- Move closer to the screen corner.
|
|
Was taking an event, when only the region coords are needed.
|
|
While only the positive axis is shown,
account for an exception when axis aligned.
|
|
|
|
Thanks to @sergey for review
|
|
|
|
Simplify the default navigation gizmo.
See: T54723
|