Bug 962409 (conninfo-widget)

[LockScreen] Implement ConnInfo widget

NEW
Unassigned

Status

Firefox OS
Gaia::System::Lockscreen
4 years ago
a year ago

People

(Reporter: snowmantw, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

To pull our the lockscreen's components from the refactoring LockScreen.
(Reporter)

Updated

4 years ago
Alias: conninfo-helper
Assignee: nobody → gweng
No longer depends on: 962408
(Reporter)

Updated

4 years ago
Blocks: 962410
(Reporter)

Updated

4 years ago
No longer blocks: 962410
(Reporter)

Updated

3 years ago
Alias: conninfo-helper → conninfo-widget
Summary: [LockScreen] Implement ConnInfo helper → [LockScreen] Implement ConnInfo widget
Created attachment 8569682 [details] [review]
[gaia] snowmantw:bug962409 > mozilla-b2g:master
(Reporter)

Comment 2

3 years ago
The patch has been pushed as a PR. Progress in the comments. TBD:

1. Write & modify tests
2. Ask QA for test as much as possible RIL cases
3. UX experiments about the 'performance'. Maybe we need some double-buffering like tricks

I would check the patch again and set feedback first.
(Reporter)

Comment 3

3 years ago
Comment on attachment 8569682 [details] [review]
[gaia] snowmantw:bug962409 > mozilla-b2g:master

Tim: before I jump into tests, I would like to patch the patch according to your opinions about how we use and fix the architecture to satisfy this use case. Beside that, I may draw some diagrams to help to clarify the states.
Attachment #8569682 - Flags: feedback?(timdream)
Comment on attachment 8569682 [details] [review]
[gaia] snowmantw:bug962409 > mozilla-b2g:master

Per discussion offline, we agreed it is no longer worthy of the risk of regression to get this landed, given the upcoming UI changes of the phone will likely change the lock screen into some other forms.

The Stream/Process libraries are good inventions and we should find somewhere else to re-purpose it.
Attachment #8569682 - Flags: feedback?(timdream)
(Reporter)

Updated

2 years ago
Assignee: gweng → nobody
You need to log in before you can comment on or make changes to this bug.