Initially the change is aimed to solve issue with line breaks being lost in projects descriptions. But it is done in a general place so line breaks behavior is more consistent all over the place.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- Restricted Diffusion Commit
rP1888f6141147: Preserve line breaks in standard remarkup fields - Required Signatures
L28 Phacility Individual Contributor License Agreement
- Write a multiline description of the project, using single \n for line breakers.
- View project details in project/view/X?
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- linebreaks
- Lint
Lint Passed Severity Location Code Message Advice src/applications/project/customfield/PhabricatorProjectDescriptionField.php:13 XHP16 TODO Comment - Unit
No Test Coverage - Build Status
Buildable 1835 Build 1836: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
Comment Actions
I'll accept this if you make the change in PhabricatorStandardCustomFieldRemarkup instead. I think we just have an odd default here. I can look at the default/general case in more detail eventually, but for now let's just swap the default to preserve.