Closed Bug 1812898 Opened 2 years ago Closed 16 days ago

[Bug] Domain highlighting and alignment

Categories

(Firefox for Android :: Toolbar, enhancement)

All
Android
enhancement

Tracking

()

VERIFIED FIXED
141 Branch
Tracking Status
relnote-firefox --- ?
firefox141 --- verified

People

(Reporter: kbrosnan, Assigned: michel)

References

(Depends on 1 open bug)

Details

(Keywords: platform-parity)

Attachments

(3 files)

From github: https://github.com/mozilla-mobile/fenix/issues/2501.

Steps to reproduce

Visit something with a subdomain, like https://bugzilla.mozilla.org.

Expected behavior

The base domain (eTLD+1) is highlighted (respectively the rest of the URL is displayed muted), as in Fennec or Desktop Firefox.
The base domain is right-aligned with the right edge of the URL bar, as in Fennec.

Actual behavior

Nothing. All the URL is displayed in the same font style, and long subdomains can push the base domain off the screen.

Compare also bug 1271998.

Device information

  • Android device: Moto G4 Play, Android 6.0.1
  • Fenix version: 1.0.9020, Build #11341807

┆Issue is synchronized with this Jira Task

Change performed by the Move to Bugzilla add-on.

Severity: -- → N/A
Assignee: nobody → michel
Depends on: 1954077
Depends on: 1954192
Depends on: 1954340
Keywords: platform-parity
Depends on: 1957105
Depends on: 1961757
Depends on: 1961829
Depends on: 1962892
Pushed by tthibaud@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/34750a3f8473 Part 1: Add domain alignment in Android toolbar component. r=tthibaud,android-reviewers https://hg.mozilla.org/integration/autoland/rev/0b308b152c3e Part 2: Enable domain highlighting in Fenix toolbar. r=tthibaud,android-reviewers
Status: NEW → RESOLVED
Closed: 16 days ago
Resolution: --- → FIXED
Target Milestone: --- → 141 Branch

For reference, here is a link to Figma referencing the behavior that is implemented on this bug. It was approved by :topotropic on the Product side.

We acknowledge that there's a Compose rewrite of the Toolbar but the decision was taken to already accept implementing that on the current one, and the same behavior will be ported to the new one.

Flags: qe-verify+

Release Note Request (optional, but appreciated)
[Why is this notable]: It changes the visible behavior of the toolbar. I believe that this is an important improvement as it make noticing phishing websites much easier.
[Affects Firefox for Android]: Yes and only affects Firefox for Android
[Suggested wording]:
[Links (documentation, blog post, etc)]:

relnote-firefox: --- → ?

Could you provide a wording for the note? thanks

Flags: needinfo?(michel)

I'm not sure how to explain this feature in a way that can be understood by non technical users.

Flags: needinfo?(michel)

Titouan, could you provide a note please? Thanks

Flags: needinfo?(tthibaud)

Release Note Request (optional, but appreciated)
[Why is this notable]: It changes the visible behavior of the toolbar. I believe that this is an important improvement as it make noticing phishing websites much easier.
[Affects Firefox for Android]: Yes and only affects Firefox for Android
[Suggested wording]: We enabled domain highlighting in the URL bar, making it clearer what is the actual website you are visiting.
[Links (documentation, blog post, etc)]:

Thanks a lot :Pascal!

Flags: needinfo?(tthibaud)
See Also: → 1969260

Note added to our Final Relnotes internal document as we don't produce pre-release notes for Android. Keeping the flag open until we add this note to our CMS in RC week, thanks.

See Also: → 1731181
See Also: → 1970209
Attached image 1812898.jpg

Verified as fixed on the latest Firefox for Android 141.0a1 from 6/5 with the following devices:

  • Samsung Galaxy Tab S9 Ultra (Android 15),
  • Samsung Galaxy S24 (Android 15), and
  • Xiaomi Mi8 Lite (Android 10).

The domain is bolded and in focus when the URL is longer - screenshots attached.
Tested on several pages: bugzilla.mozilla.org, wikipedia.org, theatlantic.com, cnn.com.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
See Also: → 1971166
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: