Add download glyph to EuiIcon

[miukimiu]

Summary

Closes #3361.

• This PR adds a download glyph to EuiIcon.
• I'm using the same arrow from the export and share glyphs. So for consistency, I did not change its size.

Dark and light themes

Design

Checklist

• Check against all themes for compatibility in both light and dark modes
• Checked in mobile
• Checked in IE11 and Firefox
• Props have proper autodocs
• Added documentation examples
• Added or updated jest tests
• Checked for breaking changes and labeled appropriately
• Checked for accessibility including keyboard-only and screenreader modes
• A changelog entry exists and is marked appropriately

[cchaos]

Are we worried that there might be conflict between the understanding of this icon and the current importAction?

They're very similar but mean very different things.

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3364/

[snide]

We discussed this today in our meeting. We decided it's OK to have them slightly different. There might be a followup to change the "export" icon to separate these concerns a little bit more.

[snide]

We discussed this today in our meeting. We decided it's OK to have them slightly different. There might be a followup to change the "export" icon to separate these concerns a little bit more.

[cchaos]

The one thing you'll want to tweak here is that the main line of the arrow cannot straddle 2 pixels. It has to align inside. This will make your icon not be fully center and 1px smaller than the whole artboard, but that is ok. The straddling will make the line very fuzzy for non-retina screens. Here's the example from the import icon

Also, don't forget to add this to the Figma EUI icon set.

[miukimiu]

@cchaos I've tweaked the icon.

Now the download icon is not centered anymore but the main line of the arrow is not straddling. Based on other similar icons I also left a 1px space below the icon.

I also added the icon to the Figma icon set and updated the PR description with new screenshots.

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3364/

[cchaos]

👍 Looks great. I checked it out on a non-retina screen too.

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3364/

[nicolapagni]

Hi,

since the icon should be merged into the master, using @elastic/eui 24.0.0 and using this type of icon I don't see that correctly.

[miukimiu]

Hi @nicolapagni,

Thank you for reporting this error. Can you please open an issue and specify with a code example how are you using the icon?

Thanks!

[nicolapagni]

@miukimiu done.

#3543