Move JS file out of new_watchlist namespace

openSUSE · Dec 2, 2022 · 2a8e72c · 2a8e72c
1 parent d46cd8e
commit 2a8e72c
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 2 deletions.
diff --git a/src/api/app/assets/javascripts/webui/application.js b/src/api/app/assets/javascripts/webui/application.js
@@ -55,7 +55,7 @@
 //= require webui/navigation.js
 //= require webui/user_profile.js
 //= require webui/long_text.js
-//= require webui/new_watchlist/collapsible_tooltip.js
+//= require webui/collapsible_tooltip.js
 //= require webui/request_show_redesign/add_review.js
 //= require webui/request_show_redesign/build_results.js
 //= require webui/request_show_redesign/rpm_lint_results.js

diff --git a/src/api/app/assets/javascripts/webui/buildresult.js b/src/api/app/assets/javascripts/webui/buildresult.js
@@ -74,6 +74,7 @@ function updateRpmlintDisplay(index) {
   });
 }
 
+// TODO: Stop using toggleBuildInfo in favor of the generic toggleCollapsibleTooltip
 function toggleBuildInfo() { // jshint ignore:line
   $('.toggle-build-info').on('click', function(){
     var replaceTitle = $(this).attr('title') === 'Click to keep it open' ? 'Click to close it' : 'Click to keep it open';

diff --git a/...ebui/new_watchlist/collapsible_tooltip.js → .../javascripts/webui/collapsible_tooltip.js b/...ebui/new_watchlist/collapsible_tooltip.js → .../javascripts/webui/collapsible_tooltip.js
diff --git a/src/api/app/views/webui/package/_buildstatus.html.haml b/src/api/app/views/webui/package/_buildstatus.html.haml
@@ -65,5 +65,4 @@
       %strong excluded/disabled
 
 :javascript
-  // TODO: call toggleCollapsibleTooltip() as soon as the new_watchlist fature is rolled out. Changing the classes in the view is also needed.
   toggleBuildInfo();