Notification icon for errors #2239
Comments
It looks like a notification now, not a popup (that was the SDK version) |
Here is the current design for notification For the layout, please refer to the measurement -> 4.3 Notification for errer, and the illustration is updated here. Just a reminder, the current fox illustration you seen on the screen is a placeholder, we are still waiting for update from Sean Martell. |
We're unable to do a popup like that in a web extension. Instead we've been using the built-in notifications. For this we'll need a small icon that will fit in the regular notifications. /cc @TLHuang @johngruen |
@ckprice will hunt down an icon, but not a blocker. |
Sorry for the late reply. May I know why we can use popup when it was on Test Pilot, but now we can only use built-in notification for Firefox Screenshots? I can provide an icon for the notification, just want to know if it's the only way we can do. |
@TLHuang Yup, we removed the SDK code and ported to a webextension, which doesn't support doorhangers |
Notification serves as a message to notify users, but it's not for error cases, especially the notification might be pretty far away from where users trigger it, so it would be inappropriate to have notification here. Is there any alternative solution or will webextension support door hanger later? If not, since we won't provide the reason why it's unavailable, I would prefer to have an unavailable badge or only disable the icon on the toolbar to imply Fx Screenshots is not able to be used at this moment. You can check the mockups below. |
When errors occur we're going to pop up a notification. We should have some error icon to go on it. I can't find much information on expected sizes. Our other icon is 32px and seems fine.
Probably just a
!
or something simple is all we need.The text was updated successfully, but these errors were encountered: