Add left and right chevrons to vanilla icon set about vanilla-framework HOT 5 CLOSED

If we have SVGs for the icons, we can add those to the icon set I guess.

from vanilla-framework.

@bartaz we do have them i'll upload them onto the assets

from vanilla-framework.

@camillet16 for Vanilla they don't need to be in assets, as we inline them in CSS.

Are there in icons drive folder (could you link those)? Or add them to comment here, and I can use the files to add to Vanilla CSS.

from vanilla-framework.

Here they are. I'm not sure if they're on a drive somewhere.

from vanilla-framework.

Is there a reason we would need to add new SVGs to the project for left/right chevrons instead of adding new left & right classes that perform the icon rotation (upstreaming the rotation work that some projects are doing)?

VF currently uses a single SVG (pointing downward) for all chevrons:

Then it rotates according to applied class (example is for p-icon--chevron-up):

Here's a link to a proposed change that doesn't need any SVGs to be changed:

Otherwise, I think we'd need to re-implement the chevron icons slightly as the vf-icon-chevron-themed mixin uses the same SVG for all directions.

Edit: See comment from Lyubo - we can use rotated chevron icon instead of using a new chevron asset

from vanilla-framework.