The default bug view has changed. See this FAQ.

[Responsive Mode] ui review

RESOLVED FIXED

Status

()

Firefox
Developer Tools
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: paul, Assigned: shorlander)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 3 obsolete attachments)

(Reporter)

Description

5 years ago
Created attachment 621064 [details]
screenshot of the current implementation

The initial mockup (attachment 619038 [details]) was a good start point. Now we know more about what feature we want to see in the first version of the Responsive Mode.

Requirements:

* we need to see the current size of the browser
* we need handles. One to resize horizontally, one to resize vertical and (optional) horizontally.
* a rotate button (or "portrait / landscape" buttons)

We also need controls for the zoom level. Changing the zoom level is needed because some resolutions won't fit in the user's screen (iPad 3 is pretty big), so we need to scale down the browser. So the size of the browser is not the "real" size anymore. The current approach is to add some zoom controls that let the user zoom in, zoom out, reset the zoom, and set the zoom to make the browser fit Firefox window. This is very confusing.

Optional: the current menulist is a simple list of sizes. It might be interesting to have a richer list that would include some icons (for example: Market place devices icons: https://marketplace-cdn.addons.mozilla.net/media/img/mkt/icons/device_icons.png)
(Reporter)

Updated

5 years ago
Blocks: 749628
(Reporter)

Updated

5 years ago
Blocks: 752857
(Reporter)

Updated

5 years ago
No longer blocks: 749628
(Reporter)

Updated

5 years ago
Assignee: nobody → shorlander
(Reporter)

Comment 1

5 years ago
I am considering removing the Zoom feature. See https://bugzilla.mozilla.org/show_bug.cgi?id=749628#c41

> The initial reason I added the ability to zoom was to be able to work with
> huge resolution (like the iPad 3 or the iPhone 4). But I was mistaken. iPad
> 3 hardware resolution has nothing to do with the resolution exposed by the
> browser. And I don't think making this tool work for huge screen resolution
> on small screen justifies adding confusing controls.
(Reporter)

Comment 2

5 years ago
Update:
- I removed the zoom feature. We now only have a menulist and a rotate button.
- the presets list is actually very small, and we might want to a couple of icons instead a menulist
  (see https://bugzilla.mozilla.org/show_bug.cgi?id=752473#c12)
(Reporter)

Comment 3

5 years ago
Created attachment 625982 [details]
screenshot of the current implementation (no zoom)

Since we removed the zoom feature, I don't think we need a new mockup. We come back to the initial idea.

I need a ui-review.

Builds will be available at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/prouget@mozilla.com-ad8943fe25ca

It needs to be preffed on (devtools.responsiveUI.enabled).
Attachment #621064 - Attachment is obsolete: true
Attachment #625982 - Flags: ui-review?(shorlander)
(Reporter)

Updated

5 years ago
Summary: [Responsive Mode] mockup for version 1 → [Responsive Mode] ui review
(Reporter)

Comment 4

5 years ago
We were considering having some more elaborated presets, but it sounds like we won't get anything soon. So I am thinking about preffing-on the tool with the current menu.

Basically, the screenshot represents precisely what I want to ship.

Comment 5

5 years ago
About the zoom option : I think it's a must have. My work laptop is 1366x768. When I work on protrait tablet apps I need to see the hole screen. Without zoom I can't :-(
(Reporter)

Comment 6

5 years ago
(In reply to hsablonniere from comment #5)
> About the zoom option : I think it's a must have. My work laptop is
> 1366x768. When I work on protrait tablet apps I need to see the hole screen.
> Without zoom I can't :-(

We still don't know how to correctly expose this feature. I don't want to bother most of the user for this specific case.

For now, no zoom feature. We need some more UX and product-related work for the zoom and dpi related functionality.
(Assignee)

Comment 7

5 years ago
Created attachment 627366 [details]
Resize Grippy
(Assignee)

Comment 8

5 years ago
Created attachment 627369 [details]
Resize Grippy - Horizontal
(Reporter)

Comment 9

5 years ago
Created attachment 627685 [details]
ui update
Attachment #625982 - Attachment is obsolete: true
Attachment #625982 - Flags: ui-review?(shorlander)
Attachment #627685 - Flags: ui-review?(shorlander)
(Reporter)

Comment 10

5 years ago
Created attachment 627686 [details]
ui update
Attachment #627685 - Attachment is obsolete: true
Attachment #627685 - Flags: ui-review?(shorlander)
Attachment #627686 - Flags: ui-review?(shorlander)
(Assignee)

Updated

5 years ago
Attachment #627686 - Flags: ui-review?(shorlander) → ui-review+
(Reporter)

Comment 11

5 years ago
Thank you Stephen.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.