Closed Bug 1691543 Opened 3 years ago Closed 3 years ago

Ensure touch mode looks good

Categories

(Firefox :: Address Bar, task, P2)

task
Points:
2

Tracking

()

VERIFIED FIXED
89 Branch
Iteration:
89.1 - Mar 22 - Apr 4
Tracking Status
firefox89 --- verified

People

(Reporter: bugzilla, Assigned: bugzilla)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-address-bar])

Attachments

(1 file)

No description provided.
Whiteboard: [proton-address-bar]
Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Iteration: --- → 89.1 - Mar 22 - Apr 4

I audited Nightly with bug 1699229 applied. Touch mode looks good. There's no change in spacing in the panel between Proton and non-Proton. The only change I'm going to make is to make the Urlbar slightly taller. In current Proton, the Urlbar is always 32px tall, so the space between the Urlbar and the edge of the toolbar increases in Touch mode. In non-Proton, we make the Urlbar 2px taller so that the spacing between the Urlbar and the edge of the toolbar is the same between Normal and Touch modes. This also has the effect of making the Urlbar a larger tap target in Touch mode.

I also tried setting the height to 36px and removing the padding-block override. I didn't think it looked very good. I like that there's slightly more padding around the Urlbar in Touch mode. It makes the UI feel roomy instead of inflated.

Pushed by htwyford@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ba57b5440804
Increase urlbar-min-height to 34px in Touch mode. r=mak
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

Touchmode was also included in the proton sign off. This was tested using a YOGA device with a windows 10 OS.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: