Closed Bug 1184841 Opened 7 years ago Closed 7 years ago

'Display as' toggle switch is misleading in app-list

Categories

(Marketplace Graveyard :: Consumer Pages, defect)

Avenir
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: trishul.goel, Assigned: frescoshashank)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:41.0) Gecko/20100101 Firefox/41.0
Build ID: 20150716004006

Steps to reproduce:

1. Goto https://marketplace.firefox.com/recommended


Actual results:

List is displayed in condensed form but the button highlighted is of expanded view.


Expected results:

The condensed list button should be highlighted in case of condensed list display
Also reproducible in `Popular` & `Categories`
Status: UNCONFIRMED → NEW
Ever confirmed: true
Shall I create another bug for `Popular` & `New`?
Flags: needinfo?(softfilebd)
No need :) you can fix all this issues in one commit and check if it can be changed globally for all :)
Flags: needinfo?(softfilebd)
Cool!
I want to work on this bug. Please assign it to me :)
Sure frescoshashank. Assigning you to this bug.

For any help you can refer to the documentation at http://marketplace.readthedocs.org/, or ask in #marketplace irc channel.

Thanks!
Assignee: nobody → frescoshashank
Status: NEW → ASSIGNED
I've completed the code changes, Please accept the pull request!

https://github.com/mozilla/fireplace/pull/1400
The UX is confusing, but when the button is grey, it is meant to mean that the button is pushed down and active. So it's intended to be that way.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Kevin the grey one always looks like inactive, if we change it I think we will get better UX. 

NI'ing Liz to opine on this.
Flags: needinfo?(ehunt)
Either way, it will be confusing. Preferably we'd change it to a slider, or just get rid of the toggle.
Flags: needinfo?(ehunt)
Slider looks better option :) Shall I log different bug for this?
Part of the confusion with the toggle is that the selected state is clickable. The selected state should NOT be clickable - only the active state (the view I want to change to) should be clickable.

Use of gray is also confusing. For another piece of work, I've suggested a lighter blue tone for the selected state. NIing Phil to get you that color - it doesn't exist yet in our palette.

A slider is not recommended at this time - we're trying to move away from this list/grid toggle altogether.
Flags: needinfo?(pwalmsley)
This will be fixed as part of MOW P1. Light blue colour for reference: #40ADFF
Flags: needinfo?(pwalmsley)
>> This will be fixed as part of MOW P1.

Phill can you please add the bug for above to this bug?
Flags: needinfo?(pwalmsley)
Flags: needinfo?(pwalmsley)
Resolution: INVALID → WONTFIX
You need to log in before you can comment on or make changes to this bug.