Page MenuHomePhabricator

Thumbnail URL guessing in MediaViewer should respect the filename abbreviation threshold
Open, LowestPublic

Description

MediaViewer tries to guess the name of the thumbnail to win a few hundred milliseconds (see getUrl here), but doesn't replace the file name with "thumbnail" when it's over the abbreviation threshold (see FileRepo::nameForThumb(). Requires exposing FileRepo::$abbrvThreshold in the filerepo API.

Priority is super low unless this turns out to be the cause of T106517.

Event Timeline

Tgr created this task.Oct 9 2015, 10:14 PM
Tgr raised the priority of this task from to Needs Triage.
Tgr updated the task description. (Show Details)
Tgr added projects: MediaViewer, good first task.
Tgr added a subscriber: Tgr.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 9 2015, 10:14 PM
Jdlrobson triaged this task as Lowest priority.Oct 13 2015, 5:01 AM
Jdlrobson set Security to None.
Tgr raised the priority of this task from Lowest to Low.Dec 7 2015, 3:57 AM
Tgr lowered the priority of this task from Low to Lowest.
Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptApr 27 2016, 4:27 PM
Tgr removed a subscriber: Tgr.Jul 9 2019, 6:05 PM