Page MenuHomePhabricator

Button to automatically copy & quote text
Closed, ResolvedPublic

Description

When replying in a differential revision or maniphest task, it might be nice to have a quote button next to each update to automatically copy & quote the text of that update into the comment box. This is fairly widely implemented on redmine & Atlassian products, as well as Bugzilla.

Event Timeline

wotte raised the priority of this task from to Needs Triage.
wotte updated the task description. (Show Details)
wotte added a project: Phabricator.
wotte added a subscriber: wotte.
epriestley triaged this task as Normal priority.Nov 18 2013, 4:26 PM
epriestley edited projects, added Remarkup; removed Phabricator.
epriestley added a subscriber: epriestley.

I think this is desirable, but we should probably include enough metadata in the quote syntax to refer to the original post.

chad added a project: Design.

I'll grab and mock out soon~ish

This could also be a link or something -- it's probably mostly blocked on the technical side since I'm hesitant to put some easy-but-ugly piece of syntax in the body, like:

{quote, src=PHID-TXCM-asd8yf98adsf98f9g}

It would probably be better if the syntax looked more like:

>>> In comment #3, @chad wrote:
> blah blah blah blah quoted text blah blah
> blah blah blah blah blah blah blah blah

...but this is slightly harder to implement if the rendering needs to know anything about the original comment, because comments are currently rendered independently. We can sort of fake this and just trust the text, but then users could do silly stuff like:

>>> In comment #666, @chad wrote:
> i am dum lol and i hate doges

That's probably fine, but it would be vaguely nice if this syntax required that the source comment and author attribution actually exist, at a minimum. Without lookups like this, we also can't do stuff like showing replies in the UI (e.g., "6 people replied to this").

None of this is intractable, but all of it tends to tread new ground on the infrastructure side of things.

@chad being "dum and hating doges" isn't so bad, it is easily falsifiable.

If y'all are going the direction of having "6 people replied to this", then you might as well just have threaded comments too.

Or at least a comment view that captured threading.

We would be totally happy if it's just plain text ;-) Thanks <3

qgil added a subscriber: qgil.
epriestley edited this Maniphest Task.
epriestley edited this Maniphest Task.
In comment T666#666, @chad wrote:

i am dum lol and i hate doges

but doges are nice :(

I'm going to close this and subtask you to maybe look at the CSS eventually since I think the styles of the "In X, Y wrote..." could probably use a bit of refinement, but it seems to work OK now.

In T4119#38, @chad wrote:
In T4119#37, @chad wrote:
In T4119#36, @chad wrote:
In T4119#35, @chad wrote:

wat

I am glad to see you are enjoying this new feature!!!!

In T4119#40, @wotte wrote:
In T4119#38, @chad wrote:
In T4119#37, @chad wrote:
In T4119#36, @chad wrote:
In T4119#35, @chad wrote:

wat

itsawesome

somuchwin

How long do you think it will be before we get a feature request to automatically fold/collapse quoted text?

How long do you think it will be before we get a feature request to automatically fold/collapse quoted text?

makeitso

epriestley edited this Maniphest Task.
In T4119#40, @wotte wrote:
In T4119#38, @chad wrote:
In T4119#37, @chad wrote:
In T4119#36, @chad wrote:
In T4119#35, @chad wrote:

wat

itsawesome
somuchwin

just testing

Extra "`" appeared in https://secure.phabricator.com/T3820#32 after quoting a comment containing three grave accents.