Closed Bug 1155534 Opened 5 years ago Closed 3 years ago

[Messages][NG] Extract NewMessage view from Conversation view

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: steveck, Unassigned)

References

Details

(Whiteboard: [p(2.2S13)=1][p(2.2S11)=5])

Attachments

(1 file)

Split thread(message) view and composer view from current Thread view. Follow up for JS/CSS separation would be needed.
Summary: [Meta][Messages][New Gaia Architecture] View panel separation for content wrapper part → [Messages][New Gaia Architecture] Separate composer view from message thread view
Whiteboard: [p=5]
Summary: [Messages][New Gaia Architecture] Separate composer view from message thread view → [Messages][New Gaia Architecture] Separate composer view from message conversation view
Depends on: 1161985
Depends on: 1161998
Whiteboard: [p=5] → [p(3=5]
Whiteboard: [p(3=5] → [p(2.2S13)=1][p(2.2S11)=5]
Duplicate of this bug: 1161999
No longer depends on: 1162030
Summary: [Messages][New Gaia Architecture] Separate composer view from message conversation view → [Messages][NG] Extract NewMessage view from Conversation view
Going to analyze Conversation --> NewMessage split deeper to define more fine grained sub-tasks.
Flags: needinfo?(azasypkin)
Duplicate of this bug: 1011085
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)
Flags: needinfo?(azasypkin)
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)
Add some thoughts in github but overall agree with the your comments about separation!
Flags: needinfo?(schung)
Thanks for your comments guys! Replied to your questions and filed several bugs for the dependencies we'll have to resolve first.
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.