angular-multiselect icon indicating copy to clipboard operation
angular-multiselect copied to clipboard

Fixes for issues - #94 #95 #92 and made path to multiselect.tmpl.html relative

Open NithinBiliya opened this issue 8 years ago • 2 comments

File changed - multiselect.js

Fix for issues -

  1. single select - button text is not mapped correctly to the string modal value #95
  2. single select - button text is not mapped correctly to the model value #94
  3. Show only one property but select complete object #92
  4. Made the path to multiselect.tmpl.html relative to multiselect.js in file multiselect.js. - Solution from -- http://stackoverflow.com/a/21103831/1298824

Changes made -

  1. Added a new function - getItemLabel(items,model)

  2. Calling the above function in function getHeaderText()

  3. get the currentScriptPath // getting the path of the multiselect.js file var scripts = document.getElementsByTagName("script") var currentScriptPath = scripts[scripts.length-1].src;

  4. Used the currentScriptPath as below - templateUrl: function (element, attr) { // multiselect.tmpl.html path relative to multiselect.js return attr.templateUrl || currentScriptPath.substring(0, currentScriptPath.lastIndexOf('/') + 1) + 'multiselect.tmpl.html'; },

Plnkr link after fixes - link3

NithinBiliya avatar Nov 22 '16 17:11 NithinBiliya

Thanks! Can you update the readme, github pages with new changes?

amitava82 avatar Nov 23 '16 08:11 amitava82

Looks like last few pulls were into master branch. gh-pages branch is not in sync with the master. Its missing few updates.

NithinBiliya avatar Nov 25 '16 10:11 NithinBiliya