Closed Bug 870879 Opened 11 years ago Closed 11 years ago

[SMS] New composer should convert <br> to \n instead of stripping spaces

Categories

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

x86
macOS
defect
Not set
normal

Tracking

(blocking-b2g:leo+, b2g18 fixed)

RESOLVED FIXED
blocking-b2g leo+
Tracking Status
b2g18 --- fixed

People

(Reporter: gnarf, Assigned: gnarf)

References

Details

Attachments

(1 file)

The new composer strips the whitespaces from the input field creating problems...

https://github.com/bocoup/gaia/pull/3#issuecomment-17731118
Depends on: 840069
Assignee: nobody → gnarf37
Attachment #748063 - Flags: review?(fbsc)
Blocks: 870626
Blocks: 870536
Attachment #748063 - Flags: review?(fbsc) → review+
Whiteboard: [NO_UPLIFT] dev-branch
blocking-b2g: --- → leo+
https://github.com/mozilla-b2g/gaia/commit/c131340c871a7c5b1182f8da0431c63da6afda4f
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [NO_UPLIFT] dev-branch → [NO_UPLIFT]
Whiteboard: [NO_UPLIFT]
v1-train: c4d4704
wrong bug ?
Flags: needinfo?(gijskruitbosch+bugs)
(In reply to Julien Wajsberg [:julienw] from comment #5)
> wrong bug ?

Yes, thanks! Sorry, mcMerge over 850 csets is a little hard to review, I must have missed this.
Flags: needinfo?(gijskruitbosch+bugs)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: