Merge pull request #105 from gatewayapps/bug/104-multiple-dots

Fixed issue with multiple periods within repo not working. Increased …
gatewayapps · Nov 28, 2017 · f5e9cf8 · f5e9cf8
2 parents 28f8536 + 5a9a27e
commit f5e9cf8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/app.js b/app.js
@@ -317,8 +317,10 @@ function ajaxRequest(type, data, url) {
 
 function addRepoToList(repoFullName, repo) {
   // add the repo to the list
-  $('.repoDropdown').append('<li data-toggle="modal" id="' + repo.replace('.', '_') + '" data-target="#kaminoModal"><a class="repoItem" href="#">' + repoFullName + '</a></li>')
-  $('#' + repo.replace('.', '_')).bind('click', () => { itemClick(repoFullName) })
+  const periodReplace = repo.replace(/\./g, '_')
+
+  $('.repoDropdown').append('<li data-toggle="modal" id="' + periodReplace + '" data-target="#kaminoModal"><a class="repoItem" href="#">' + repoFullName + '</a></li>')
+  $('#' + periodReplace).bind('click', () => { itemClick(repoFullName) })
 }
 
 function populateUrlMetadata() {

diff --git a/css/style.css b/css/style.css
@@ -36,7 +36,7 @@ a {
     z-index: 1000;
     display: none;
     float: left;
-    min-width: 260px;
+    min-width: 280px;
     height: 300px;
     overflow-y: auto;
     overflow-x: hidden;

diff --git a/manifest.json b/manifest.json
@@ -35,7 +35,7 @@
         "webNavigation",
         "*://github.com/*"
     ],
-    "version": "2.7",
+    "version": "2.8",
     "web_accessible_resources":[
         "icons/*.png",
         "bootstrap/js/bootstrap.min.js",