Closed
Bug 1062321
Opened 11 years ago
Closed 11 years ago
[Utility Tray] When notifications overflow they should be scrollable
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:2.1+, b2g-v2.1 verified, b2g-v2.2 verified)
People
(Reporter: epang, Assigned: cwiiis)
References
Details
(Keywords: regression, Whiteboard: [systemsfe])
Attachments
(2 files)
|
319.81 KB,
image/jpeg
|
Details | |
|
46 bytes,
text/x-github-pull-request
|
mikehenrty
:
review+
fabrice
:
approval-gaia-v2.1+
|
Details | Review |
notifications.jpg
When there are more notifications then the allotted space the notification should become drag-gable between the header and system component. See attached image
|
Assignee | |
Comment 1•11 years ago
|
||
draggable confused me when I saw this e-mail, retitling :)
Pretty sure this is a regression from the swipe notification handling, probably easily fixed. Won't assign just yet though, want to sort out a couple of other bugs first.
Keywords: regression
Summary: [Utility Tray] When notifications overflow they should be drag-gable → [Utility Tray] When notifications overflow they should be scrollable
|
|
Assignee | |
Comment 2•11 years ago
|
||
Assignee: nobody → chrislord.net
Status: NEW → ASSIGNED
Attachment #8483544 -
Flags: review?(mhenretty)
|
|
Assignee | |
Comment 4•11 years ago
|
||
[Blocking Requested - why for this release]: A duplicate bug of this was a blocker (bug 1061213)
blocking-b2g: --- → 2.1?
|
||
Updated•11 years ago
|
Target Milestone: --- → 2.1 S4 (12sep)
|
|
||
Comment 6•11 years ago
|
||
Comment on attachment 8483544 [details] [review]
Fix notification overflow not being scrollable
Works like a charm.
Attachment #8483544 -
Flags: review?(mhenretty) → review+
|
|
Assignee | |
Comment 7•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
status-b2g-v2.1:
--- → affected
status-b2g-v2.2:
--- → fixed
Resolution: --- → FIXED
|
|
Assignee | |
Comment 8•11 years ago
|
||
Comment on attachment 8483544 [details] [review]
Fix notification overflow not being scrollable
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined: Can't scroll notifications when you have more than can fit in the panel
[Testing completed]: Tested locally, mhenretty confirms it works, merged to master
[Risk to taking this patch] (and alternatives if risky): Low risk of other issues when swiping on notifications
[String changes made]: None
Attachment #8483544 -
Flags: approval-gaia-v2.1?
|
||
Updated•11 years ago
|
Attachment #8483544 -
Flags: approval-gaia-v2.1? → approval-gaia-v2.1+
|
||
Comment 9•11 years ago
|
||
|
|
||
Comment 10•11 years ago
|
||
this is verified and works on Flame 2.2 Master KK (319mb) (Full Flash) and 2.1 KK (319mb) (Full Flash)
Flame 2.2 Master KK (319mb) (Full Flash)
Device: Flame 2.2 Master
BuildID: 20141011040204
Gaia: 95f580a1522ffd0f09302372b78200dab9b6f322
Gecko: 3f6a51950eb5
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2 Master)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0
2.1 KK (319mb) (Full Flash)
Environmental Variables:
Device: Flame 2.1 KK (319mb) (Full Flash)
Build ID: 20141010000201
Gaia: d71f8804d7229f4b354259d5d8543c25b4796064
Gecko: 7fa82c9acdf2
Version: 34.0a2 Flame 2.1 KK (319mb)
Firmware Version: v180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
|
|
||
Comment 11•11 years ago
|
||
In Flame 2.2 Master KK (319mb) (Full Flash) and 2.1 KK (319mb) (Full Flash), with multiple notifications the list is scrollable, both up and down with no errors.
|
||
Updated•11 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
|
|
Assignee | |
Updated•11 years ago
|
|
|
Assignee | |
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•