Icon needed for navigation not loading #763
Replies: 4 comments 1 reply
-
Sure, we can add this to the backlog. Thanks for adding the delivery date requirement and placing the request well in advance 🙇 Another option is always to contribute to Paste. There are guidelines on how to add icons to the system https://paste.twilio.design/icons/how-to-add-an-icon |
Beta Was this translation helpful? Give feedback.
-
Hey @johannaweintraub, is this icon planned on being used anywhere else? |
Beta Was this translation helpful? Give feedback.
-
Not at the moment
… On Sep 16, 2020, at 1:37 PM, corinne-nagel ***@***.***> wrote:
Hey @johannaweintraub, is this icon being used anywhere else?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Beta Was this translation helpful? Give feedback.
-
Ok, we will use the grey colored error icon as we don't want to alarm users
with the red. I agree the use of that icon will help tie it back to the
alert. I will update the empty state to white. Sorry, I was looking at an
old spec I believe. You can close this out. Thanks
…On Thu, Sep 17, 2020 at 9:01 AM corinne-nagel ***@***.***> wrote:
Since this wouldn't be an icon used in other places we wouldn't suggest
adding it to Paste; we ideally are only adding systemized icons. If you
think you need to use the light switch icon, we may suggest you just doing
a one-off addition yourself through this:
https://github.com/twilio-labs/svg-to-react
After looking at the full design, the team thinks it might make more sense
to add the error icon in this state instead of adding an entirely new icon.
The visual cue of using the same error icon will help users relate the
alert above to the error in the navigation. Also, if you could match the
styles you're using with the empty state shown here to keep our patterns
consistent, that would be great!
https://share.goabstract.com/b72d29bc-9dbf-4f99-9a52-d3b4e6bf53cd?collectionLayerId=1a0fdec9-70cc-4303-93c9-6e3495196b94&mode=design
Hope this helps!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#763 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABPUMJSYU7GHPCLN25JJLVDSGIXFNANCNFSM4RPG26ZA>
.
|
Beta Was this translation helpful? Give feedback.
-
[email protected]
Could this icon please be added to Paste? Mid-October is the required date.
https://share.goabstract.com/920337aa-b45a-4b5e-b67d-2336a69f1617
Beta Was this translation helpful? Give feedback.
All reactions