4 commits

Author	SHA1	Message	Date
joakin	68f04d62ec	Wire settings dialog to show on SETTINGS_SHOW action ... Now when on a hovercard and clicking the cog, it should show the hovercards settings menu. Additional changes: * Change the changeListener/settings to create and attach the DOM element just once to the DOM Change-Id: Id685ddcda9532528fc62b383539788f785089ef0	2016-12-13 14:05:43 +01:00
joakin	4d5269320f	Introduce the settings dialog UI ... * Port ext.popups.desktop/ext.popups.settings.js to settingsDialog.js * Blank tests for now. Needs Qunit integration tests. * Transform into a factory function for future testing. * Saving functionality is commented out, will be removed when immplemented in the actions * Add new incomplete action saveSettings * Will perform the saveSettings async tasks and then trigger enabling or disabling popups. * Rename action settingsDialogClosed to hideSettings for consistency Change-Id: I3936d3a4bc476de16d76025139be09f1798796c4	2016-12-13 11:42:21 +01:00
Jhernandez	2fa5b9ef02	Revert "Settings dialog" ... This reverts commit 047bccfac1. It created merge conflicts with a bunch of patches that were days old and about to be merged. Instead of rebasing 10 patches, let's revert this one, finish merging the other ones and just rebase and fix this one. I'll re-introduce the patch in a bit. Change-Id: Ib495755b0ab4bfa5fdf5590b1271792862a47d4b	2016-12-12 12:54:33 +00:00
Jeff Hobson	047bccfac1	Settings dialog ... Also removes some unused (redundant) actions and the renderer reducer. Bug: T152223 Change-Id: I878c7f16d71f8cdbd74a47ffeb9dadc4decf2883	2016-12-12 04:54:34 -05:00