[Icons]Core Icon Revision

[Reqrefusion]

#13840

[maxwxyz]

IMO the wireframe and hidden line icon is still too similar.
Maybe the hidden line icon should indicate the triangulation?

[Reqrefusion]

IMO the wireframe and hidden line icon is still too similar. Maybe the hidden line icon should indicate the triangulation?

It probably mixes it up more. I think changing the perfect made it much, much easier to understand. Also I don't understand this idea. Also, I looked at other programs and it usually happens like this in all of them.

[maxwxyz]

So you say this makes it easier to understand but you don't understand it? The hidden lines display the triangulated state as the geometry is rendered with Coin3D. IMO not very useful for working with solids but it's the great for meshes.

[Reqrefusion]

So you say this makes it easier to understand but you don't understand it? The hidden lines display the triangulated state as the geometry is rendered with Coin3D. IMO not very useful for working with solids but it's the great for meshes.

Ok I won't lie. I never looked at this view because I thought it was similar to other CAD programs. Which I don't normally use. I've never used mesh or anything like that. Now I kneel before the greatness of FreeCad. Indeed, this appearance has turned into something useful that can be used.

The downside was that I misunderstood and had to remake the entire group to make their perspectives more understandable. But it was definitely worth it. They became much more understandable.

[maxwxyz]

I like it now. Thanks.
Please remove src/Gui/Icons/edge-selection.svg this icon from the PR as this will be deleted, otherwise this will get conflicts.
Then please squash your commits.

[maxwxyz]

The resizing broke several alignments to the pixel grid so the smaller icons appear blurry.
I've attached some screenshots to the before/after. As mentioned, the elements and strokes should be aligned to the pixel grid to prevent aliasing and scale better.
In Inkscape you can view the grid under View -> Page grid.

src/Gui/Icons/accessories-text-editor.svg this icon is a good example how it should be, as everything is aligned.

@kadet1090 @obelisk79 FYI

[kadet1090]

Yes, pixel alignment is crucial for the icons to render properly and be sharp so it must be fixed before merging.

[Reqrefusion]

Done, ready to merge.

[chennes]

@maxwxyz could you please approve (or not, as the case may be!) the change you requested?

[maxwxyz]

@kadet1090 @obelisk79 could you check as well and comment?

[chennes]

@obelisk79 if you would like to comment please do so, otherwise we will merge next week.

[Reqrefusion]

I don't think this eye looks disproportionate. But I still revert the change.
I made the changes you requested. Do you have another request? @obelisk79