Fix for not saving new preset settings

ejci · Jul 12, 2015 · f56a7ef · f56a7ef
1 parent 635a6ae
commit f56a7ef
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/extension/common/presets.js b/extension/common/presets.js
@@ -1,5 1,5 @@
 var presets = (function() {
- 'use strict';
  'use strict';
  var list = [];
  var selected = false;
 

diff --git a/extension/popup/popup.js b/extension/popup/popup.js
@@ -177,8 177,9 @@ var init = function(prs) {
  switch (ev.target.value) {
  case 'action::save':
  modal.confirm('Do you want to save "'   selected.name   '" preset?', function() {
- for (var i = 0; i < selected.gains.length; i )
- selected.gains[i] = getValue('ch-eq-slider-'   (i   1))
  for (var i = 0; i < selected.gains.length; i ) {
  selected.gains[i] = getValue('ch-eq-slider-'   (i   1));
  }
  console.log('action::save', selected);
  presets.setPreset(selected);
  });
@@ -188,8 189,13 @@ var init = function(prs) {
  modal.prompt('New preset name', function(name) {
  if (name && name.length > 0) {
  var preset = JSON.parse(JSON.stringify(selected));
  for (var i = 0; i < preset.gains.length; i ) {
  preset.gains[i] = getValue('ch-eq-slider-'   (i   1));
  }
  preset.name = name;
- presets.setNewPreset(preset)
  presets.setNewPreset(preset);
  presets.setSelected(name);
 
  }
  }, function() {
  });

diff --git a/readme.md b/readme.md
@@ -1,6 1,6 @@
 #Audio EQ for Chrome
 
-With [Audio EQ for Chrome](https://chrome.google.com/webstore/detail/lfafdlnjaliaghpjdajmlcnnblkgcefh/null) you can control your audio and video from all tabs in one place.
 With [Audio EQ for Chrome](https://chrome.google.com/webstore/detail/lfafdlnjaliaghpjdajmlcnnblkgcefh/) you can control your audio and video from all tabs in one place.
 More info [here](http://lab.ejci.net/Chrome-Audio-EQ/).
 
 For best results install HTML5ify (https://chrome.google.com/webstore/detail/html5ify/jikbjpjgjmmdhcmlagappehlpiljoaop).