3 commits

Author	SHA1	Message	Date
joakin	e976af78e2	Hygiene: #processLinks Only use global variables at the edges ... Instead of defaulting the config to the global mw.config when the param is not defined, always pass it in (it is called just once in application code), that way there's no need to test for the optional argument behavior and the function is pure. Change-Id: Ib1addb3060826f58dce2d6f928252ce1888a4293	2016-11-30 17:01:49 +00:00
Sam Smith	0d300867ab	Make API request after 500 ms ... If the user has abandoned the link or dwelled on a different link, then don't make the API request. Changes: * Fix a bug in the linkDwell reducer where the activeLink was always being set to undefined. * Add the private fetch action creator. * Make the linkDwell action dispatch the result of the fetch action creator after 500ms. Supporting changes: * Make the link* action creators take DOM elements rather than jQuery instances so that: 1. Testing whether the state tree has changed is an identity comparison. 2. jQuery instances are constructed only when required. * Document the ext.popups.Gateway type. * Document the Redux.Store type. * Rename the "previews-page-title" data attribute to "page-previews-title" to avoid confusion. Change-Id: I0b1cf3337a6f8d6450ad2bd127cb292ebb73af4f	2016-11-17 13:20:11 +00:00
Sam Smith	9611d3b2db	Add LINK_DWELL and LINK_ABANDON actions ... Supporting changes: * Add mw.popups.processLinks. * Extract the existing unit tests for mw.popups.selectPopupElements and mw.popups.getTitle. * Fix Grunt QUnit timeout. Change-Id: I325bcb15abc6e0b745d78b7308a346a034ab2988	2016-11-10 11:47:55 +00:00