[GitHub] [accumulo] milleruntime commented on a diff in pull request #2674: Bug fix for caption in tservers table in monitor

[GitBox <gi...@apache.org>]

milleruntime commented on code in PR #2674:
URL: https://github.com/apache/accumulo/pull/2674#discussion_r864724654


##########
server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/tservers.js:
##########
@@ -18,29 +18,25 @@
  */
 "use strict";
 
-var tserversList;
+var tserversTable;
 /**
  * Creates tservers initial table
  */
 $(document).ready(function() {
-    
+
+  // hide the note about highlighted rows by default
+  $('#recovery-caption').hide();

Review Comment:
   Should be able to hide this on the FTL file. If you do it here, every time the page loads, you may see it flash on the screen. The Manager page (manager.ftl) has an alert that is hidden by default using `style="display: none;"`



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org