@Jeroen-Bakker reply to your review.
Draw manager should be as functionality agnostic as much as possible. Adding this if-statement might be ok as we also check if bone overlay is enabled.
…
I don't think it's necessary to iterate over bases, so let's get rid of this:
bool selected = false;
LISTBASE_FOREACH (Base *, base, BKE_view_layer_object_bases_get(view_layer)) {
…
I'll keep an eye out what people say once it's merged with main, especially about the selection occlusion when set to 0.
It might be a good idea to make the selection occlusion optional, but…
@fclem says it perfectly. I don't use this feature, so I don't have a strong opinion on how it should work. Definitely a design issue rather than a bug.
I have no jurisdiction to review but as the guy who opened the issue I want to say: this looks good to me 👍
I'd say change Nederlandse taal
to just Nederlands
.
@JulienKaspar Thanks for your feedback!
The selection is also occluded if the retopology_offset is set to 0. That should maybe not be the case.
I don't see why not. The slider is just for…
@mano-wii Replying doesn't seem to work properly so I'll just comment here.
To your first comment about localized comments and grease pencil objects:
I was thinking of doing that too, but…
Re-requesting review from @mano-wii because I made several changes. They're all based on your feedback from the last review, and I don't expect you to find any problems, but just in case.