Make Polygon use fill="currentColor"

[felquis]

On this line:

primitives/packages/react/arrow/src/Arrow.tsx

Line 28 in 4ee9d36

{props.asChild ? children : <polygon points="0,0 30,0 15,10" />}

It would be helpful if it uses fill="currentColor" cause I could apply color directly on <Popover.Arrow className="text-gray-primary" />

Also, SVG fill=currentColor is very wide spread without @react-icons/all-icons which make it kind of the expected behavior of most SVGs.

What do you think?

[benoitgrelard]

Hey @felquis, thanks for raising this.

Whilst it makes sense for icons as they are usually rendered inline along text (color/currentColor), that isn't the case here for the arrow as it doesn't live in a textual context.

That's why I think it makes more sense to style using fill.
With tailwind you can use fill-blue-500 for that.