Page MenuHomePhabricator

Reduce excessive ChartWizard preview requests
Closed, ResolvedPublic

Description

The chart wizard sends a preview (parse) request after every change (e.g. typing each character of the title).

Requests can also complete out of order, allowing an older response to replace the preview for the current chart definition.

We should add debouncing to help limit how frequently automatic previews are requested, and we should also ensure that only the latest preview (parse) api response updates the preview.

Event Timeline

Change #1309785 had a related patch set uploaded (by Aude; author: Aude):

[mediawiki/extensions/Chart@master] Add debounce to the chart preview requests

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

Change #1310044 had a related patch set uploaded (by Aude; author: Aude):

[mediawiki/extensions/Chart@master] Add SourceDataPreview and RenderedChartPreview components

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

Change #1309785 merged by jenkins-bot:

[mediawiki/extensions/Chart@master] Add debounce to the chart preview requests

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

Change #1310044 merged by jenkins-bot:

[mediawiki/extensions/Chart@master] Add SourceDataPreview and RenderedChartPreview components

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