Change are fine. Just check the braces to match current code style.

Seems to be ok as it only changes defaults.

Code style: Formatting

Added TODO comment

@Clément Foucault (fclem) I couldn't find a 100% good solution for the show_vcol parameter.
For now I propose to check in the draw manager if it is active and if so it will pass the show_vcol via a parameter. I didn't like to add show_vcol to the PBVH as that would need the draw manager modify that setting. I also didn't like adding this to the depsgraph evaluation as it scatters drawing code to the depsgraph. Do you have an idea to make it more clean?

Fixed issue when a workbench material is used multiple times,
but not all objects have a vertex color. They shared the same
hash, but that would draw the objects using the incorrect
shaders.

change to is_uniform_color.
Test why vertex colors are not showing when opening older sculpt blend files

Optional generating vcol vertex data

Don't show overlay when an modifiers are enabled in edit mode

Code style + cleanup

@Clément Foucault (fclem) can you take a special look into the naming convention inside the draw_cache?

Code style + cleanup

Support for sculpt drawing

Test commit: sculpt drawing

Support Sculpting drawing

Rebased with latest master.
Still todo: sculpting drawing

I discussed with @Bastien Montagne (mont29). We changed the logic to work within a grid_flow and a column_flow. Also cleaned up the code as it fitted better on the original code. When you have time please review!

clang-format

Added and tested logic for column_flow

I will eventually render 5-6 items, so the alignment is not really nice. I expected that even_columns or even_rows would take care of this. For example when rendering 5 items all items will be rendered as a fully horizontal and vertical aligned.

Support to render on grid_flow

Missed one

Solved comments from code review

@Brecht Van Lommel (brecht) can you review this?

Split when expanding horizontal and not rendering UI_BTYPE_ROW

With D4694 the basics of dynamic paint seems to work

Rebased with master (clang-format)

Rebased with master (clang-format)

@Pablo Vazquez (pablovazquez) came with the idea to use icons in stead of text? The two line will not always look nice like

We will show on a single line the next options:

Seems to be that for armatures that are draw in front this is the case. I bisected this back until first of February this year.
@Clément Foucault (fclem) Is there a technical limitation for in front rendering? We support it for xray and normal rendering

I can confirm that the import is failing on current b280.

Have you checked if it isn’t a Out of memory issue (ram)

Hi Jonathan,