Page MenuHomePhabricator

Phabricator task URLs don't enjoy hover text
Closed, DuplicatePublic

Description

Upstreamed as https://secure.phabricator.com/T5378

If I write http://fab.wmflabs.org/T413 in this comment you can't hover it to get its title: I must write T413 (happened in http://fab.wmflabs.org/T52#28 ).

Bugzilla was smarter and did both, <s>see e.g. https://bugzilla.wikimedia.org/show_bug.cgi?id=32940#c2 </s> (no longer works, because the url in the comment at old-bugzilla points to another subdomain)

Details

Reference
fl416

Event Timeline

flimport raised the priority of this task from to Lowest.Sep 12 2014, 1:40 AM
flimport set Reference to fl416.

aklapper wrote on 2014-06-15 12:45:44 (UTC)

Feels like WONTFIX to me. Bugzilla has other places where it fails to provide any hover text (e.g. InlineHistory bug links to the same Bugzilla installation) so it's not much better.

Nemo_bis wrote on 2014-06-15 20:51:07 (UTC)

Bugzilla has other places where it fails to provide any hover text (e.g. InlineHistory bug links to the same Bugzilla installation)

That works for me. If you happen to have a bug link handy I'd be interested.

aklapper wrote on 2014-06-15 22:24:25 (UTC)

In T416#8, @Nemo_bis wrote:

That works for me. If you happen to have a bug link handy I'd be interested.

For example the thrid line after comment 1 in https://bugzilla.wikimedia.org/show_bug.cgi?id=43151#c1 says "See Also: http://bugzilla.wikimedia.org/show_bug.cgi?id=2089" and mouseover does not provide any popup info. I guess the InlineHistory extension does not check whether "See Also" URLs start with the local instance. Popup in InlineHistory works for "Blocks" and "Depends on" though because that's always the local instance. :)

Nemo_bis wrote on 2014-06-16 05:14:59 (UTC)

Thanks for the example. Still, I don't see how it matters: Phabricator has no metadata fields and hence no inline history for them, so the only case to handle is comments rendering.

Quiddity updated the task description. (Show Details)Dec 1 2014, 2:43 AM
Quiddity set Security to None.
Quiddity updated the task description. (Show Details)Dec 1 2014, 2:45 AM
Quiddity updated the task description. (Show Details)
In T253#799348, @Qgil wrote:

This proposal conflicts with T76095: Pasting Phabricator task URLs into descriptions should create pretty link. Which one should go forward?

That task is just a specific solution (for I don't know which problem), this is a correctly formulated task which focuses on goals.

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptMay 18 2016, 9:18 PM