If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Use .SVG images for empty state of about:pages

RESOLVED WORKSFORME

Status

()

Firefox for Android
Logins, Passwords and Form Fill
RESOLVED WORKSFORME
3 years ago
a year ago

People

(Reporter: antlam, Unassigned, Mentored)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
> It'd be great to try using svg, because we've talked about doing that but
> haven't done so yet. We don't use one large resource that can get scaled
> down because (I assume) it's more memory and CPU intensive for smaller,
> likely less powerful devices. I wonder if SVG has those setbacks? Or perhaps
> they're just larger files that we have to include.

This is something we talked about in bug 1101746, filing this bug to track that work in an effort to close the other and get things moving. Also, Chenxia brought up some issues that we might run into so I thought we should split up this work.
(Reporter)

Comment 1

3 years ago
Created attachment 8575515 [details]
icon_key_emptypage_SVG.svg

Here's the SVG, Ally. I made it slightly bigger than the spec, but none of the margins change. 

I also went with the Key here to keep it consistent with the door hanger and toolbar icon. I'll attach .PNG versions in the other bug just in case.
Flags: needinfo?(ally)

Updated

3 years ago
Blocks: 1141898
Assignee: nobody → ally
Flags: needinfo?(ally)
relevant https://etherpad.mozilla.org/svg-guidelines
Assignee: ally → nobody
Mentor: ally@mozilla.com, michael.l.comella@gmail.com, liuche@mozilla.com
Has this bug been resolved already? I see that the about:logins page already uses the SVG icon.

Or are there any other icons that need to be replaced? Thanks!
Flags: needinfo?(alam)
(Reporter)

Comment 4

a year ago
I'm not too sure actually. Ally (who was working on this) is no longer with Mozilla so I'm not able to confirm. If you're saying we already do this then this can be closed :)
Flags: needinfo?(alam)

Comment 5

a year ago
I think we can close this out.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.