Closed
Bug 1155534
Opened 10 years ago
Closed 8 years ago
[Messages][NG] Extract NewMessage view from Conversation view
Categories
(Firefox OS Graveyard :: Gaia::SMS, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: steveck, Unassigned)
References
Details
(Whiteboard: [p(2.2S13)=1][p(2.2S11)=5])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
Details |
Split thread(message) view and composer view from current Thread view. Follow up for JS/CSS separation would be needed.
Reporter | ||
Updated•10 years ago
|
Summary: [Meta][Messages][New Gaia Architecture] View panel separation for content wrapper part → [Messages][New Gaia Architecture] Separate composer view from message thread view
Updated•10 years ago
|
Blocks: sms-sprint-2.2S11
Whiteboard: [p=5]
Reporter | ||
Updated•10 years ago
|
Summary: [Messages][New Gaia Architecture] Separate composer view from message thread view → [Messages][New Gaia Architecture] Separate composer view from message conversation view
Updated•10 years ago
|
Blocks: sms-sprint-2.2S13
Whiteboard: [p=5] → [p(3=5]
Updated•10 years ago
|
Whiteboard: [p(3=5] → [p(2.2S13)=1][p(2.2S11)=5]
Updated•9 years ago
|
Summary: [Messages][New Gaia Architecture] Separate composer view from message conversation view → [Messages][NG] Extract NewMessage view from Conversation view
Comment 2•9 years ago
|
||
Going to analyze Conversation --> NewMessage split deeper to define more fine grained sub-tasks.
Flags: needinfo?(azasypkin)
Comment 4•9 years ago
|
||
Hey guys, This PR is just copy of conversation.js file, but it provides good way to discuss split on method by method basis :) I've added comments/propositions for almost every piece that we may want to leave in conversations, move to new_message.js or move to somewhere else. It would be great to have your opinion on that as well, so that we can slowly start splitting these views. Thanks!
Flags: needinfo?(schung)
Flags: needinfo?(felash)
Updated•9 years ago
|
Flags: needinfo?(azasypkin)
Comment 5•9 years ago
|
||
Added more information where I thought it was needed, but otherwise mostly agree with you. Github seems to have a hard time with this PR though :(
Flags: needinfo?(felash)
Reporter | ||
Comment 6•9 years ago
|
||
Add some thoughts in github but overall agree with the your comments about separation!
Flags: needinfo?(schung)
Comment 7•9 years ago
|
||
Thanks for your comments guys! Replied to your questions and filed several bugs for the dependencies we'll have to resolve first.
Comment 8•8 years ago
|
||
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•