Closed Bug 1291573 Opened 8 years ago Closed 2 years ago

Unified autocomplete/autofill "Visit" row should show the page title if it is in the history or bookmark

Categories

(Firefox :: Address Bar, enhancement, P3)

enhancement
Points:
5

Tracking

()

RESOLVED FIXED
103 Branch
Tracking Status
firefox103 --- fixed

People

(Reporter: emk, Assigned: daisuke)

References

Details

(Keywords: papercut, Whiteboard: [fxsearch][snt-triaged][search-urlbar])

Attachments

(4 files)

I'm often frustrated the absence of the page title after unified autocomplete has been introduced. For example, when I type "bug" in the URL bar, the Action row is "bugzilla.mozilla.org - Visit". It should be something like "bugzilla.mozilla.org - Visit Bugzilla Main Page".
The first reason we don't show the title, is that in many cases we may not have it. The autoFill entry is generated on the fly, when we know the given domain is known to Places (but probably we only have title for a subpage). The second reason is that this entry should come as fast as possible while the user is typing, we try to select the bare minimum for that to avoid delays. that said it's something that could be evaluated, if it has a value from an UX point of view, we could look at the perf problem and see if it's acceptable to fetch a title WHEN we have a title. On the other side, the right fix is likely bug 1235397, but it's not trivial to do.
Blocks: 1262507
Depends on: 1235397
Priority: -- → P3
Whiteboard: [fxsearch]
(In reply to Marco Bonardo [::mak] (Away 6-20 Aug) from comment #1) > The second reason is that this entry should come as fast as possible while > the user is typing, we try to select the bare minimum for that to avoid > delays. It is OK for me to show the "Visit" row without the page title first, then add the page title when the query returns the result.
(In reply to Masatoshi Kimura [:emk] from comment #2) > It is OK for me to show the "Visit" row without the page title first, then > add the page title when the query returns the result. that's hard to do for code architectural reasons.
Type: defect → enhancement
Priority: P3 → P5
Severity: normal → S3
Points: --- → 5
Keywords: papercut
Priority: P5 → P3
Summary: Unified autocomplete "Visit" row should show the page title if it is in the history or bookmark → Unified autocomplete/autofill "Visit" row should show the page title if it is in the history or bookmark
Severity: S3 → N/A

May be related to autofilling full URLs, not only origins, since currently autofill fills only origins. May also need UX input.

Whiteboard: [fxsearch] → [fxsearch][snt-triaged][search-urlbar]
Attached image with-visit-label.png (deleted) —

I tried to show the title. I attached the screenshot.
I thought it looked a bit confusing, because the "Visit" label and the title got mixed.
Next, I will attach another screenshot without the Visit label.

Attached image without-visit.png (deleted) —

This one seems simpler and easier to understand at least for me.
What do you think? As Drew said, we may better ask UX team.

Flags: needinfo?(adw)
Flags: needinfo?(VYV03354)

But, if we choose the above, how about the page not visited yet.
Will we show "URL - Visit" as the same as now?
It may make confusion a bit since there may be pages whose title is “Visit”.

I prefer no "Visit" label.

Flags: needinfo?(VYV03354)

Thanks for testing these Daisuke. I agree with Masatoshi -- of the two options, without "Visit" is better. However, it's interesting because the screenshot in comment 7 shows the URL and title in the opposite places they usually are. Of course it makes sense because it's an autofill result, and autofill results show their URL first. But I wonder if it should be styled like a typical history result when a title is available.

So instead of this:

https://bugzilla.mozilla.org/ -- Bugzilla Main Page

Do this:

Bugzilla Main Page -- https://bugzilla.mozilla.org/

What do you think? Could you try that please and see how it looks? We should probably get Ryan's and/or Henry's opinion too.

Flags: needinfo?(adw)
Attached image title-first.png (deleted) —

I attached a screenshot that shows the title first.
This also looks good!

One point of concern is when the URL has not yet been visited.
Perhaps the following suggestions could be chosen:

In either case, the position that shows URL will be changed like Title - URL (visited) or URL - Visit (not visited yet).
There may be some discomfort with the change in location?

As you said, I'll ask Ryan as well.

(In reply to Daisuke Akatsuka (:daisuke) from comment #12)

I attached a screenshot that shows the title first.
This also looks good!

One point of concern is when the URL has not yet been visited.
Perhaps the following suggestions could be chosen:

In either case, the position that shows URL will be changed like Title - URL (visited) or URL - Visit (not visited yet).
There may be some discomfort with the change in location?

As you said, I'll ask Ryan as well.

Thanks @Daisuke, some thoughts below:
I also like the latest iteration on order with "title" : "url-string"

  • More consistent with suggestions below (Drew's point)
  • I think the title is more "human" (more legible, easier to scan) than a URL
    In the case of a not yet visited URL - I think having the "-Visit" appended still makes sense as it offers context for the user as to the action to take/why they are being shown this URL. My concern otherwise is that without that context it could be confusing as to why it's shown, especially if a user has never visited said URL before.

I do think we should get Product (Henry) buy-in on this as well.

Thank you very much for your clear answer, Ryan!
I'll ask Henry as well.

Hello Henry!
There is the idea that shows the page title for the autofill result row.
Could you please confirm the way to display the information?
Thanks!

Flags: needinfo?(hmoseti)

(In reply to Daisuke Akatsuka (:daisuke) from comment #14)

Thank you very much for your clear answer, Ryan!
I'll ask Henry as well.

Hello Henry!
There is the idea that shows the page title for the autofill result row.
Could you please confirm the way to display the information?
Thanks!

I agree with the last agreed iteration with Ryan & Drew's input above. Also lets monitor feedback from users to measure the impact. Thanks.

Flags: needinfo?(hmoseti)

Thank you very much, Henry!
I will proceed with this.

Assignee: nobody → daisuke
Status: NEW → ASSIGNED
Attachment #9279386 - Attachment description: WIP: Bug 1291573: Show page title for autofill result. → Bug 1291573: Show page title for autofill result.
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
Flags: qe-verify+
Flags: in-testsuite+
Blocks: 1791657
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: