Change FxA avatar rendering from crisp to auto

RESOLVED FIXED in Firefox 56

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: rfeeley, Assigned: lyret, Mentored)

Tracking

({good-first-bug})

49 Branch
Firefox 56
good-first-bug
Points:
---

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(2 attachments)

(Reporter)

Description

2 years ago
Created attachment 8887997 [details]
crisp-edges.png

The Firefox Accounts profile picture renders crisply, but should be on auto.

Make sure this doesn't affect border on hover/active states.
CSS property Ryan is talking about: https://developer.mozilla.org/en-US/docs/Web/CSS/image-rendering
Mentor: eoger
Keywords: good-first-bug

Comment 2

2 years ago
Hi,

I would like to take this as my first bug. Please mark the assignee to me. Thanks!
Hi,

We typically assign a bug to a contributor once a patch has been posted.

Comment 4

2 years ago
Hi Edouard,

I get it. I'll try. Thank you.
Comment hidden (mozreview-request)
Attachment #8889243 - Flags: feedback?(eoger)

Comment 6

2 years ago
mozreview-review
Comment on attachment 8889243 [details]
Bug 1382290 - Change Fxa avatar rendering from crisp-edges to auto.

https://reviewboard.mozilla.org/r/160302/#review165552

This looks great thanks!
I also verified the appMenu and it looks like the avatar rendering is done right.

::: commit-message-5928d:1
(Diff revision 1)
> +BUG 1382290 - Change Fxa avatar rendering from crisp-edges to auto

- BUG -> Bug
- Add r?eoger at the end

=> Bug 1382290 - Change Fxa avatar rendering from crisp-edges to auto. r?eoger
Attachment #8889243 - Flags: feedback?(eoger) → feedback+
(Assignee)

Updated

2 years ago
Attachment #8889243 - Attachment description: BUG 1382290 - Change Fxa avatar rendering from crisp-edges to auto → Bug 1382290 - Change Fxa avatar rendering from crisp-edges to auto. r?eoger
You need to change the actual commit message ;)

Comment 8

2 years ago
Pushed by eoger@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/36d1a7817bc0
Change Fxa avatar rendering from crisp-edges to auto. r=eoger
I fixed the bug message and landed this for you. Thanks a lot for contributing!
Comment hidden (mozreview-request)
(Assignee)

Comment 11

2 years ago
Sorry for the delay eoger and thanks for the fix!

Comment 12

2 years ago
mozreview-review
Comment on attachment 8889243 [details]
Bug 1382290 - Change Fxa avatar rendering from crisp-edges to auto.

https://reviewboard.mozilla.org/r/160302/#review165920
Attachment #8889243 - Flags: review?(eoger) → review+
Assignee: nobody → lyretandrpg

Comment 13

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/36d1a7817bc0
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
You need to log in before you can comment on or make changes to this bug.