Page MenuHomePhabricator

Use EditorWidget for edit post
Closed, ResolvedPublic


The summary editor and the topic/post content editor (i.e. not creating new topics/replies, but editing existing ones) come to mind. We should also audit to make sure the old editor code is unused, and then remove it.

Edit post:

Topic summary: was extracted into T111102: Use EditorWidget for topic summary

Event Timeline

Catrope raised the priority of this task from to Needs Triage.
Catrope updated the task description. (Show Details)
Catrope added a subscriber: Catrope.

For testing - to check after the fix:

  • different wikitext icons for 'Reply' and 'Edit reply'

Screen Shot 2015-08-18 at 11.40.37 AM.png (551×776 px, 62 KB)

Change 234423 had a related patch set uploaded (by Mattflaschen):
Initial refactoring to create ContentWidget

Change 234603 had a related patch set uploaded (by Sbisson):
[WIP] Edit post widget

SBisson renamed this task from Port remaining editor uses to EditorWidget to Use EditorWidget for edit post.Sep 1 2015, 8:10 PM
SBisson updated the task description. (Show Details)

Checked in beta - edit/new reply posts are ok:

Screen Shot 2015-09-02 at 2.12.09 PM.png (629×1 px, 81 KB)

DannyH added a subscriber: DannyH.

Change 234423 merged by jenkins-bot:
Refactor CAPTCHA support into data model and widget