From f2a696a3c477d6c9c91ee55ce7c8bcc68e8f3bac Mon Sep 17 00:00:00 2001 From: SHUBHAM SHARMA Date: Tue, 14 Sep 2021 20:01:46 +0530 Subject: [PATCH] fixes console error module selector (#1543) Co-authored-by: Rishabh Shukla <42492389+blurry-x-face@users.noreply.github.com> Co-authored-by: Harsh Khandeparkar <34770591+HarshKhandeparkar@users.noreply.github.com> Co-authored-by: Jeffrey Warren --- examples/lib/defaultHtmlSequencerUi.js | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/examples/lib/defaultHtmlSequencerUi.js b/examples/lib/defaultHtmlSequencerUi.js index 7283c95c..3e4d6034 100644 --- a/examples/lib/defaultHtmlSequencerUi.js +++ b/examples/lib/defaultHtmlSequencerUi.js @@ -17,7 +17,6 @@ function DefaultHtmlSequencerUi(_sequencer, options) { // look up needed steps from Url Hash: function importStepsFromUrlHash() { var hash = urlHash.getUrlHashParameter('steps'); - if (hash) { _sequencer.importString(hash); _sequencer.run({ index: 0 }); @@ -27,9 +26,7 @@ function DefaultHtmlSequencerUi(_sequencer, options) { function selectNewStepUi() { var m = $(addStepSel + ' select').val(); - //if so that whenever the module dropdown is empty it will not show error it will take first value. - if(!m) m = arguments[0]; - else $(addStepSel + ' .info').html(_sequencer.modulesInfo(m).description); + if(m) $(addStepSel + ' .info').html(_sequencer.modulesInfo(m).description); $(addStepSel + ' #add-step-btn').prop('disabled', false); }