Changes requsted except subpanel as subpanel depth is a problem, also they can't be reordered if they become subpanels.

In D10505#279580, @Aaron Carlisle (Blendify) wrote:

Was there a reason you moved the draw_modifier functions to there own function?

Apologies for delay, I updated the patch @Aaron Carlisle (Blendify)

Requested change done, except 2 for reasons.

Apologies for delay, here's the new diff with required changes, except the two mentioned for reasons

In D10505#270786, @Aaron Carlisle (Blendify) wrote:

If you want to look at the modifiers now, a lot of them need updates to add layout.use_property_split and decorators

In D10505#270590, @Aaron Carlisle (Blendify) wrote:

I made suggestions for some overall changes, next we can have a look at modifiers.

In D10517#269303, @Brecht Van Lommel (brecht) wrote:

Switching between presets does not work well with this change.

• More and more modifiers are added, it should clear existing ones (and I guess re-add the default Samplig modifier).
• All preset should modify the same properties like chaining, use_same_object, alpha, ... . Where they are not set now they should set the default value.

@Brecht Van Lommel (brecht) @Aaron Carlisle (Blendify) required changes done ✔️

@Aaron Carlisle (Blendify) what changes should be made next?

@Brecht Van Lommel (brecht) made required changes

Presets modify active linestyle and no longer creates new data-blocks

In D10517#265766, @Brecht Van Lommel (brecht) wrote:

Right, adding new datablocks is not the way other presets work, that should be avoided.

@Aaron Carlisle (Blendify) I made requested changes, only subpanel change wasn't done, there are already many subpanels inside each one, also they should be reorderable, it is a key feature for the update

In D10517#265199, @Aaron Carlisle (Blendify) wrote:

I think sticking with the current solution is best for now.

In D10505#264854, @Aaron Carlisle (Blendify) wrote:

d. Line style breadcrumb, making sure that editing line style setting always on the correct line style

This looks a bit weird with no icon between the text.

In D10517#265160, @Aaron Carlisle (Blendify) wrote:

Was looking into the comments about adding presets, I don't know if this is possible with the complexity of modifiers.

Rename Thick Joins to Thick Vertices

In D10517#265160, @Aaron Carlisle (Blendify) wrote:

Was looking into the comments about adding presets, I don't know if this is possible with the complexity of modifiers.

In D10505#265086, @Aaron Carlisle (Blendify) wrote:

Create a new revision

In D10505#265086, @Aaron Carlisle (Blendify) wrote:

Create a new revision

Regarding VIEWLAYER_PT_freestyle_linestyle subpanels, we tried that earlier on the prototype, it was a disaster. making Freestyle extremely hard to use and read. harder to read than the original UI

In D10505#264848, @Aaron Carlisle (Blendify) wrote:

I think the reason this was neglected is no one expected Freestyle to stick around long, however, a replacement is still far from.

It might take a little to review such a large patch, but can you split the new presets into a separate patch.
With the preset patch, it would be nice to include examples.
The presets can be merged a lot easier this way as they do not require a strict review.

so here's some possiblities
1)they are using a cutom build of Blender

2. it is likely they are using commandline rendering

maybe rendering your project using commandline and see if it produces good or bad result
https://docs.blender.org/manual/en/latest/advanced/command_line/render.html

In T68687#882150, @Brecht Van Lommel (brecht) wrote:

@Iyad Ahmed (iyadahmed2001), it's best to create a new report then with example .blend file and system information, to be sure we are looking at the same problem.

I still see the problem in 2.82, it occurs whenever there are negative numbers as input for both Vector Math and Math nodes modulo, EEVEE seems to be more accurate, it is the Cycles implementation that doesn't produce desirable results.