Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16556 )

Change subject: Upgrade Bootstrap to 3.4.1 in web UI
......................................................................


Patch Set 2:

(1 comment)

> Patch Set 1:
>
> Do we want/need to non `min` versions of the files?

We don't need them, as per the discussion about npm.js, I removed these as well.

http://gerrit.cloudera.org:8080/#/c/16556/1/www/bootstrap/js/npm.js
File www/bootstrap/js/npm.js:

PS1:
> I guess it's desirable to import as less as possible to keep it functional,
Fair enough, removed it.



--
To view, visit http://gerrit.cloudera.org:8080/16556
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0b92241457af2997b3486781dc5e9e5f9e971471
Gerrit-Change-Number: 16556
Gerrit-PatchSet: 2
Gerrit-Owner: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 08 Oct 2020 06:44:16 +0000
Gerrit-HasComments: Yes

Reply via email to