Fixed issue with multiple periods within repo not working. Increased …

…width of repo dropdown
gatewayapps · Nov 28, 2017 · cdf8a0c · cdf8a0c
1 parent 28f8536
commit cdf8a0c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 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() {
@@ -388,6 +390,7 @@ function openDropdown() {
 }
 
 function itemClick(repo) {
+  console.log('item clicked: ' + repo)
   // add the item to the most used list
   addToMostUsed(repo)
 

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;