vdiravka commented on a change in pull request #1594: DRILL-6942: Provide
ability to sort list of profiles on Drill Web UI
URL: https://github.com/apache/drill/pull/1594#discussion_r248649875
##########
File path: exec/java-exec/src/main/resources/rest/profile/list.ftl
##########
@@ -109,6 +110,21 @@
float:left
}
</style>
+
+<style>
Review comment:
It can be segregated by comments. Also looks like the general `CSS to
control DataTable Elements` comment from the first block is related to second
too.
Nevertheless I found two same CSS blocks in `options.ftl` and `profile.ftl`.
Does it make sense to create CSS file and link it in these FTL files?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
