• Just checked the getSourceControlBaseRevision() functions

I honestly still do not understand what is happening here, what does it mean disabled:3:1:E043 or error:3:1:E019 or anything, but:

Hi @Bukkit can you please publish F281219? thanks :)

And, just to clarify, have MediaWiki something similar? Probably the "Mute User". Is Mute user the needed thing?

We could expand a bit the Task description to clarify why the Disable User button and the Remove User are not sufficient for this purpose.

I think that D25359 is a nice example of a complete use case, where one Diff from one repo depends from another Diff from another repo.

I noticed that Phorge exports the date in this format:

Nice. Let's file a Task for that. We can work with arc patch D25359 --skip-dependencies in the meanwhile

looks like arc patch tries to patch D25358 first, because this diff "Depends on" that one - but that one is in arcanist, not phorge.

Can you share a nice A/B test on Microsoft Windows to upload?

Premising that I never manually "use" the scrollbar: I always use the mouse roll on desktop, or the finger swipe on mobile.

git rebase master

Soft+1 but let's wait for another +1. Thanks!

+1 with just that inline suggestion

Tested, no nuclear implosions, thanks as usual :)

Soft-accept! Tested and it works. Waiting for another tester.

Soft-accept. Thanks! I will test this a bit over additional integer fields

Remove unrelated Credentials change now covered in T15580 instead

Handle empty $value in PhabricatorStandardCustomFieldPHIDs.php instead

In D25346#10708, @aklapper wrote:

In D25346#10087, @avivey wrote:

I think this should be implemented using the getHead() method that's already defined.

Could you elaborate a bit, please? I'm not sure what exactly not to do here, and what to do instead. Thanks in advance!

clicking such a link w/javascript disabled will do nothing. Changing it to #123 and no JS will do something....

Implicitly, Phorge generally works fine for browsers where Javascript is completely disabled.

4. Why these nice URLs are set in these links, if nobody visit them?

Less priority of course, since this Task I authored is super-stupid

In D25346#10087, @avivey wrote:

I think this should be implemented using the getHead() method that's already defined.

OK I've done my best :D I admit it helped me psychologically.

If this is a feature request, then add all the things needed for a feature request - as in, "why would this feature be needed" and "what problem you're trying to solve".

Is this a bug or a feature request?

Why would it need to be a that?
Is this a bug or a feature request?

OK this was easy, but it really cannot be improved more than this in Legalpad or Phame. And this deserves a dedicated fix.

I'm 99% sure that this only affects Phame and Legalpad. I tested in Audit, Maniphest, Paste, Pholio and Files without issues.

This situation could be improved even more doing:

Increasing priority to normal since this feature is supposed to help to collaborate with others, but at the moment you only collaborate with yourself as default

Should this be documented here? I think yes

I'm mentioning this small thing in the Changelog since some people may have legacy Dashboards with a Panel that was Archived just to try. So there is less surprise on your Phorge homepage.

fix comment

LGTM, except for the comment text.

Yeah just try "arc land" and, if it does not work, we can land this for you

Okay, now how do I land this? Just arc land? Again, in my current setup there is no way of unit testing, so I don't know if it is possible...

Optimizes querying of project edges of multiple documents.

In T15540#12393, @avivey wrote:

I don't think it's related ...

Also maybe Macro app could deserve a nice Hovercard :)