Status

P1
normal
RESOLVED FIXED
3 years ago
4 months ago

People

(Reporter: jryans, Assigned: hholmes)

Tracking

Trunk
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox46 affected)

Details

(Whiteboard: [multiviewport] [ux] [mvp-rdm])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Refer to the user story in bug 1172309.

Currently this work includes a list of popular devices as well as a way to edit the list.

If the initial list presented is just "popular" devices, we should think about how (if at all) we use the data from other devices in our device DB.  Do we only allow "promoting" others from the DB to the main list, or can you free form enter an entirely new custom device?
(Reporter)

Updated

3 years ago
Blocks: 1241714
(Reporter)

Updated

3 years ago
Blocks: 1241720

Updated

3 years ago
Flags: qe-verify-
Ryan, is this bug to handle a) the UX of promoting devices from one list to another via some sort of settings, or b) the creation of the list itself?
Flags: needinfo?(jryans)
(Reporter)

Comment 2

3 years ago
(In reply to Helen V. Holmes (:helenvholmes) (:✨) from comment #1)
> Ryan, is this bug to handle a) the UX of promoting devices from one list to
> another via some sort of settings, or b) the creation of the list itself?

So far, we've just been making on UX bug per user story, so I meant for this to encompass all the points in bug 1172309.  (We can certainly make more UX bugs if we want to!)

From the current story text, I am imagining we need at least:

* A list of devices somewhere that we can choose from to apply their attributes
* Any changes (if we are planning any, we may not be) to the overall UI when a device is selected vs. a random size
* Some way to change the list

It's not super clear to me from the story what "change the list" means.  Did we agree to only support "promoting" to a default list from our larger DB, or is there also addition of user-defined devices as well?

Maybe Bryan can help, I can't quite remember.
Flags: needinfo?(jryans) → needinfo?(clarkbw)
You can definitely file more UX bugs if you want to breakdown the UX work further.
(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #2)
> It's not super clear to me from the story what "change the list" means.  Did
> we agree to only support "promoting" to a default list from our larger DB,
> or is there also addition of user-defined devices as well?
> 
> Maybe Bryan can help, I can't quite remember.

This is going to depend a bit on what helen is thinking for the layout of the UX.  For the initial version I think we can support only "promoting" a device to the default list.
Flags: needinfo?(clarkbw)

Updated

3 years ago
Priority: -- → P2

Updated

3 years ago
Priority: P2 → P1

Updated

3 years ago
Assignee: nobody → hholmes
Status: NEW → ASSIGNED
Iteration: --- → 47.1 - Feb 8

Updated

3 years ago
Iteration: 47.1 - Feb 8 → 47.2 - Feb 22
Created attachment 8717982 [details]
device-list-ux-notes.png

The viewport should have a select box at the top with a curated list of devices. (I've put a few suggestions in my designs which I'll transcribe at the bottom, although they're not backed by any data yet.)

The last option in the select should be "Edit list..." which pops up a modal if selected.

The modal has all of the devices from Jan's device list CDN, excluding the devices that have been marketed as Firefox OS devices. In the future, we should be able to make changes to the CDN and have the RDM modal/lists update pretty easily.

The 'x' on the modal closes the modal without saving the user selections.

Hitting 'done' closes the modal, saving the user selections. 

The select box should display the devices alphabetically.

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
See Also: → bug 1241714

Updated

3 years ago
Whiteboard: [multiviewport][ux] → [multiviewport] [ux] [mvp-rdm]

Updated

4 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.