Page MenuHomePhabricator

Get rid of artifical click delay in MediaViewer
Closed, ResolvedPublic0 Estimated Story Points

Description

Navigation on clicks leading away from the current page is artifically delayed in MediaViewer to protect EventLogging requests. Now that EventLogging supports sendBeacon, this should no longer be necessary.

Event Timeline

Tgr raised the priority of this task from to Needs Triage.
Tgr updated the task description. (Show Details)
Tgr added projects: MediaViewer, Analytics.
Tgr added a subscriber: Tgr.
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
This comment was removed by Tgr.
This comment was removed by Tgr.
gerritbot added a subscriber: gerritbot.

Change 190703 had a related patch set uploaded (by Gergő Tisza):
Fix share tabs

https://gerrit.wikimedia.org/r/190703

Patch-For-Review

Nevermind, wasn't originating from MediaViewer. I removed the preventDefault-based click delay though.

Change 190705 had a related patch set uploaded (by Gergő Tisza):
Remove delay when logging link clicks

https://gerrit.wikimedia.org/r/190705

Patch-For-Review

Gilles moved this task from Untriaged to Reviewed, needs improvements on the Multimedia board.
Gilles added a subscriber: Gilles.

This task ought to be renamed.

Gilles edited a custom field.
Tgr renamed this task from Get rid of sync AJAX calls in MediaViewer to Get rid of artifical click delay in MediaViewer.Feb 18 2015, 8:05 PM
Tgr updated the task description. (Show Details)

Change 190705 merged by jenkins-bot:
Remove delay when logging link clicks

https://gerrit.wikimedia.org/r/190705