VOOZH about

URL: https://bugzilla.mozilla.org/show_bug.cgi?id=1811873

⇱ 1811873 - Regression: The border of the new back/Forward indicator looks to too pixelated/not smooth


Closed Bug 1811873 Opened 3 years ago Closed 3 years ago

Regression: The border of the new back/Forward indicator looks to too pixelated/not smooth

Regression: The border of the new back/Forward indicator looks to too pixelated/not smooth
Firefox
General
Trunk
Unspecified
macOS
defect
Points:
---
RESOLVED FIXED
RESOLVED
FIXED
112 Branch
Iteration:
---
a11y-review
Accessibility Severity
Performance Impact
Webcompat Priority
Webcompat Score
Tracking Status
firefox-esr102 --- unaffected
firefox109 --- unaffected
firefox110 --- unaffected
firefox111 --- wontfix
firefox112 --- fixed
Tracking Status
relnote-firefox
firefox-esr102
firefox-esr115
firefox-esr140
firefox-esr153
firefox109
firefox110
firefox111
firefox112
firefox152
firefox153
firefox154
---
QA Whiteboard:
---
Has STR:
---
Change Request:
---
Bug Flags:
Signature:
None
This bug is publicly visible.

 
👁 Image
pixelated border.png
346.33 KB, image/png
Details
👁 Image
Nightly_vs_Target.png
37.08 KB, image/png
Details
👁 Image
The swipe-to-nav icon
1.21 KB, image/svg+xml
Details
👁 Image
Bildschirmfoto 2023-01-25 um 09.32.42.png
6.37 KB, image/png
Details
👁 Image
More fine-grained icon
8.90 KB, image/svg+xml
Details
👁 Image
Bildschirmfoto 2023-01-25 um 23.40.12.png
60.79 KB, image/png
Details
👁 Image
stroke-width=1 icon
8.98 KB, image/svg+xml
Details
48 bytes, text/x-phabricator-request
Details | Review
👁 Image
Swipe feedback.svg
800 bytes, image/svg+xml
Details
👁 Image
new_vs_old.png
402.90 KB, image/png
Details
Reporter

Description

3 years ago
Attached image pixelated border.pngDetails

Nightly 111.0a1 (2023-01-23) (64-Bit)
macOS 12.6.2

1.) Open a new Tab
2.) Visit a page in that tab
3.) Swipe back with the Trackpad to the previous page

Actual: the white outline looks too pixelated/not smooth.

Expected: A nice and smooth border.

This is a regression with the new new blue back/forward indicators in Nightly.

A screenshot is attached.

Reporter

Comment 1

3 years ago

Mozregression brings me to bug 1799563.

Blocks: 1799563
Keywords: regression
Regressed by: 1799563

Set release status flags based on info from the regressing bug 1799563

:hiro, since you are the author of the regressor, bug 1799563, could you take a look? Also, could you set the severity field?

For more information, please visit auto_nag documentation.

Flags: needinfo?(hikezoe.birchill)
Assignee

Updated

3 years ago
OS: Unspecified → macOS

Mehmet, would you mind trying this binary to see whether it's still pixelated or not? You can download mac's binary from the target.dmg link in the "Artifacts and Debugging Tools" pane. I don't expect too much it fixes the pixelated case though.

Flags: needinfo?(mehmet.sahin)
Reporter

Comment 4

3 years ago
Attached image Nightly_vs_Target.pngDetails

(In reply to Hiroyuki Ikezoe (:hiro) from comment #3)

Mehmet, would you mind trying this binary to see whether it's still pixelated or not? You can download mac's binary from the target.dmg link in the "Artifacts and Debugging Tools" pane. I don't expect too much it fixes the pixelated case though.

Hi Hiroyuki, I think it was better before. Now the border is too bright and it highlights much more the pixels (please see the attachment). If it is too hard or not worth to fix, I think it should stay then as it is with the current more greyish border.

Thanks :)

Flags: needinfo?(mehmet.sahin)
Attached image The swipe-to-nav iconDetails

Hmm, then it seems like it's just a matter of the size.

Can you please open this attachment in Firefox without any zooming? I expect it's still pixelated.

Flags: needinfo?(hikezoe.birchill) → needinfo?(mehmet.sahin)
Reporter

Comment 6

3 years ago

(In reply to Hiroyuki Ikezoe (:hiro) from comment #5)

Created attachment 9313969 [details]
The swipe-to-nav icon

Hmm, then it seems like it's just a matter of the size.

Can you please open this attachment in Firefox without any zooming? I expect it's still pixelated.

It looks good?! Please see the screenshot without any zooming.

Flags: needinfo?(mehmet.sahin)

I can't say it looks better. :/ (I did look it with zooming, since pngs aren't scaled by zooming.)

Attached image More fine-grained iconDetails

Is this better?

Flags: needinfo?(mehmet.sahin)
Reporter

Comment 9

3 years ago

(In reply to Hiroyuki Ikezoe (:hiro) from comment #8)

Created attachment 9314133 [details]
More fine-grained icon

Is this better?

Hmm, the previous icon with the more bolder white border from comment 5 looked better I think. The rounding is more smoother compared to the new "more fine-grained". The new "more fine-grained" looks still too pixelated. Attached side-by-side attached to a window border.

Flags: needinfo?(mehmet.sahin)
Attached image stroke-width=1 iconDetails

Josh, is it okay for you to increase the semi-circle's line width from 0.5 to 1.0? It looks like it's much better in terms of pixelation.

Attaching SVG is the width=1.0 icon. It was originally exported as SVG from an icon in the spec doc you wrote, and I changed the width from 0.5. So maybe it was just caused by the Figma exporting tool.

Flags: needinfo?(jberman)

Hi,

Yes I am on board, it definitely looks better at 1.0

Thank you!

Flags: needinfo?(jberman)
Severity: -- → S3
Priority: -- → P3

Thank you Josh!

Also use a more fine-grained SVG.

Assignee: nobody → hikezoe.birchill
Status: NEW → ASSIGNED

:hiro this is the final week of 111 nightly, 111 goes to beta next week.
Do you plan on getting this reviewed and landing in central this week?

Flags: needinfo?(hikezoe.birchill)

Yes, I'd hope someone in desktop-theme-reviewers reviews this change within the time frame.

Flags: needinfo?(hikezoe.birchill)

From a review comment;

! In D168009#5579660, @dao wrote:
What's up with this huge <path>? Looks like it's almost setting pixels rather than leveraging the strength of vector graphics? The fact that this is inlined into the browser window markup makes this seem particularly unsound. From just looking at what kind of image we're trying to create here, can't we get there using basic shapes like circles and lines?

UX folks are usually pretty good at creating good vector graphics. Could you please reach out to get some help with this?

Josh, can you please give us another version of the swipe-to-navigation icon SVG file with basic shapes for Mac?

Honestly with my very limited SVG knowledge, it would be a bit tricky since it's not full circle.

Flags: needinfo?(jberman)
Attached image Swipe feedback.svgDetails

Connected with Tati - another designer who is more proficient with visual work. She was able to pull the asset into illustrator and fine tune it and export from there. Let me know if this works please

Flags: needinfo?(jberman)
Attachment #9314386 - Attachment description: Bug 1811873 - Increase the stroke-width of the swipe-to-navigation icon on Mac. r?#desktop-theme-reviewers → Bug 1811873 - Replace the swipe-to-navigation icon on Mac. r?dao

Thank you Josh for the quick update! I wonder why the new SVG size got shrunk 1px, but it's not a big deal.

Assignee

Updated

3 years ago
See Also: → 1818360
Pushed by hikezoe.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/82b41f86e346 Replace the swipe-to-navigation icon on Mac. r=dao
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Reporter

Updated

3 years ago
Flags: needinfo?(hikezoe.birchill)
Reporter

Comment 21

3 years ago
Attached image new_vs_old.pngDetails

Hello, I tested it in latest Nightly 112.0a1 (2023-02-24) (64-Bit) on Retina Mac. The border still looks too pixelated :(

On a non-retina Mac it looks more worse :(

Maybe the indicator is too small and should be larger?

BTW: What was wrong with the good large grey indicator? Why it has to be replaced? The new indicator is a downgrade I think :(

A screenshot is attached. Thanks for checking.

You need to log in before you can comment on or make changes to this bug.