Old
New
VDG |
Old
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
Visually I think this is fine and a good improvement.
I wonder why your SVGs have two seperate paths as opposed to unifying them into one path (as I usually do)? Just curious if there is any benefit to using one approach above the other.
Also, judging by the MDN Page I don't think the fill-rule="evenodd" does anything here.
Sometimes the optimizers prefer to group rather than combine. These days, optimization is used to keep diffs readable so I can find mistakes and to cut down the size most of the way. Optimizing down to the last byte we can get rid of is not necessary.