HomePhabricator

DNM Do not load RevisionSlider initially, add a button to show/hide it
4dbcda3bc4bdUnpublished

Authored by WMDE-leszek on Aug 5 2016, 4:14 PM.

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

DNM Do not load RevisionSlider initially, add a button to show/hide it

Instead of loading RevisionSlider only add a little button
to expand RevisionSlider on top of the diff page.

This makes RevisionSlider only steal a bit of space over the
diff, and only inserts quite a big slider to users that want
to have it visible for the particular diff.

API calls are only made once RevisionSlider has been expanded.

This is re-submit of b0f229d75f905a9570497dc095b410f20f4b53ae
that was reverted in I26427faaa00b38c2aa1377a66224c9062dcca302

Bug: T141871
Change-Id: I879de5774b2cce7b908e73cbbe869fd48d6afa23

Details