Open
Conversation
Contributor
Contributor
Author
I didn't think of that, but it is possible, one would then have to specify that the gradient is applied before the rotation - I don't know if that will cause weird effects for other cases. I'll forward that idea internally. But since the gradient haven't worked as intended for years (or decades) I think just removing them as in this PR would work as well; they cannot be that important. BTW: The idea was to rotate both of the polygons, and I hope we agree that something is needed. |
Contributor
Right, I missed the cylinder.
Yes, the current look is dreadful :) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Split off from #4711
This is for the Modelica.Fluid.Examples.AST_BatchPlant.BaseClasses.TankWith3InletOutletArraysWithEvaporatorCondensor model used in Modelica.Fluid.Examples.AST_BatchPlant and is for the slanted pipe at the top:
This PR intends to get it to look as follows (this is also how it looks like in Dymola 2026x):
Supporting gradients on polygons would instead give (notice the odd gradients for the top-pipe; to me this just looks odd):
I assume the idea was something like the following (using paint) - but that requires gradients along the axis of polygons, which seems quite difficult to support: