Closed
Bug 824164
Opened 8 years ago
Closed 8 years ago
change in layout of chat menu
Categories
(Firefox Graveyard :: SocialAPI, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 23
People
(Reporter: mixedpuppy, Assigned: mixedpuppy)
References
Details
(Keywords: uiwanted)
Attachments
(3 files, 8 obsolete files)
For better visibility we should change the location of the nub to be immediately to the left of the chat windows, rather than being left justified. If that is not possible, moving it next to sidebar would at least connect it more with the provider. requesting ux/design input
Assignee | ||
Comment 1•8 years ago
|
||
quick patch to show change of location for chat nub
Assignee | ||
Comment 2•8 years ago
|
||
image showing new location of nub
Assignee | ||
Updated•8 years ago
|
Whiteboard: [needs-ux]
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #702087 -
Attachment is obsolete: true
Assignee | ||
Comment 4•8 years ago
|
||
Attachment #702089 -
Attachment is obsolete: true
Attachment #738717 -
Flags: ui-review?(jboriss)
Comment 5•8 years ago
|
||
Putting the overflow nub on the left is a good way to handle this, but let’s style the nub’s background identically to the top of chat windows themselves. After all, both are the “collapasible” handles, editable via cursor click, to a larger portion of a network's content. I can r+ this bug just the styling - items of how the nub behaves and displays should be looked at as well. Not able to reproduce the nub in the current build - should it be in there?
Assignee | ||
Comment 6•8 years ago
|
||
Assignee | ||
Comment 7•8 years ago
|
||
Assignee | ||
Comment 8•8 years ago
|
||
fixes nub color per irc with boriss about the suggested change in the mockup. The primary issue was the transparency on the nub image by default. The goal is to match the button appearance with the minimized chat window appearance.
Assignee: jboriss → mixedpuppy
Attachment #738716 -
Attachment is obsolete: true
Attachment #740468 -
Attachment is obsolete: true
Attachment #740471 -
Attachment is obsolete: true
Attachment #740499 -
Flags: review?(mhammond)
Comment 9•8 years ago
|
||
Attachment #740181 -
Attachment is obsolete: true
Comment 10•8 years ago
|
||
Attachment #740510 -
Attachment is obsolete: true
Comment 11•8 years ago
|
||
(trying again, getting weird Bugzilla errors today) After talking through this on IRC and giving it some though, I'm attaching a new mockup with some tweaks to the previous one. The mockup shows the nub in "inactive" color state by default, while "active" state only when the user is actively engaging with the menu. This seems like the most consistent way to handle the coloring, and also means that a collapsed chat tab of the same height uses the same color scheme as the button rather than appearing "less active" than it. I've also included two views of the menu that comes from the nub - one as a "goal" state consistent with Australis styling, and one as a default placeholder if that's too difficult to achieve in time for next code freeze.
Attachment #740512 -
Attachment is obsolete: true
Comment 12•8 years ago
|
||
Comment on attachment 740499 [details] [diff] [review] chat nub location.patch I get the impression from Boriss's later comments that this patch will still need further tweaks, so clearing review request - let me know if I misunderstood.
Attachment #740499 -
Flags: review?(mhammond)
Assignee | ||
Comment 13•8 years ago
|
||
(In reply to Mark Hammond (:markh) from comment #12) > Comment on attachment 740499 [details] [diff] [review] > chat nub location.patch > > I get the impression from Boriss's later comments that this patch will still > need further tweaks, so clearing review request - let me know if I > misunderstood. We're not handling the australis styling in this bug, the followup for that is bug 866172.
Assignee | ||
Comment 14•8 years ago
|
||
Comment on attachment 740499 [details] [diff] [review] chat nub location.patch Either Mark or Felipe (since it's now Marks weekend)
Attachment #740499 -
Flags: review?(mhammond)
Attachment #740499 -
Flags: review?(felipc)
Comment 15•8 years ago
|
||
Comment on attachment 740499 [details] [diff] [review] chat nub location.patch Review of attachment 740499 [details] [diff] [review]: ----------------------------------------------------------------- LGTM - clearing Felip's request as you asked for either
Attachment #740499 -
Flags: review?(mhammond)
Attachment #740499 -
Flags: review?(felipc)
Attachment #740499 -
Flags: review+
Assignee | ||
Comment 16•8 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=106393db34d5
Assignee | ||
Comment 17•8 years ago
|
||
don't try before the first coffee. here's a new one https://tbpl.mozilla.org/?tree=Try&rev=a635f2d95737
Assignee | ||
Comment 18•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f0b7c3e3b622
Comment 19•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f0b7c3e3b622
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Assignee | ||
Updated•8 years ago
|
Attachment #738717 -
Flags: ui-review?(jboriss)
Updated•2 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•