mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ross Allen" <ross...@mesosphe.re>
Subject Re: Review Request 12395: Replace bootstrap.js with angular-ui.js
Date Fri, 12 Jul 2013 21:45:32 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12395/
-----------------------------------------------------------

(Updated July 12, 2013, 9:45 p.m.)


Review request for mesos.


Changes
-------

Update Makefile.am.


Repository: mesos


Description
-------

Angular UI Bootstrap[1] duplicates all Bootstrap plugins in Angular directives. The only Bootstrap
plugins currently used are Popovers and Modals, and so they are replaced with their Angular
UI cousins here.

Note: Extended Popover behavior from popovers.js, which allowed only one popover open at a
time, is not duplicated here. Popovers will be replaced rather than extended here.

[1] http://angular-ui.github.io/bootstrap/


Diffs (updated)
-----

  src/Makefile.am 5d6f309c877ed37bb530c9c22e274c52fb72b7d4 
  src/webui/master/static/app.js 404516f8c760cdff564a9b1e8d2e0d74407daf67 
  src/webui/master/static/bootstrap/js/bootstrap-alert.js fa0806ea1a3d411b17dbe6a1c5d3eec3721ebd48

  src/webui/master/static/bootstrap/js/bootstrap-button.js a9e6ba7b10734938de2eb4a2352a88d7eb58c546

  src/webui/master/static/bootstrap/js/bootstrap-carousel.js 96e5a8191175dda7e56edf49223caedd29692582

  src/webui/master/static/bootstrap/js/bootstrap-collapse.js d02f6fda4f5f67b451cbe5ea9a238242e9ad14d7

  src/webui/master/static/bootstrap/js/bootstrap-dropdown.js ec0588dc1aa3862743ba495a6761e2d547ee6013

  src/webui/master/static/bootstrap/js/bootstrap-modal.js c831de6b64b532a7d75c92444ac674f744d78c30

  src/webui/master/static/bootstrap/js/bootstrap-popover.js d5ecfa92077ceeda50569b17c98bffe16b2b2600

  src/webui/master/static/bootstrap/js/bootstrap-scrollspy.js 4946ee9360006681386104035ba89201af8164f0

  src/webui/master/static/bootstrap/js/bootstrap-tab.js 88641de864c0d68cffda561b6e181c73b42b3a0d

  src/webui/master/static/bootstrap/js/bootstrap-tooltip.js 577ead48b56eeb08e2caa4f88bd8fc3da009654b

  src/webui/master/static/bootstrap/js/bootstrap-transition.js 7e29b2fd0198ae318327239c227f9303f6675760

  src/webui/master/static/bootstrap/js/bootstrap-typeahead.js 95a0fcdb78357d6c6e7e51739810ec3e7421adf6

  src/webui/master/static/bootstrap/js/bootstrap.js 7244954107bdec606d759bb0c2d11c1e6c305562

  src/webui/master/static/bootstrap/js/bootstrap.min.js 8c06421e105c24445e14ed5d2a4322e2e06a3c41

  src/webui/master/static/controllers.js f9d7c978fe4cf6d2fbb4d45990b277e44485f0cd 
  src/webui/master/static/framework.html c3827e886351308dadd61fbe6a7570ad5d3d5110 
  src/webui/master/static/frameworks.html acb0eaa3ca8e893c92f4cfe887760d8a6f3ad182 
  src/webui/master/static/home.html 0074f318e87070fd2f5004f23a4d80d04a7047d1 
  src/webui/master/static/index.html 066ca37a3d091a4a4731f0119f84a1197a71c4c8 
  src/webui/master/static/popovers.js 28666ddb1987eeb6159ff167c2c28d7b2cfd7d8a 
  src/webui/master/static/slave.html 9a52f907271e362d3dac127fc74e8c4a8581e43d 
  src/webui/master/static/slaves.html 508748f16c87a1ad2f3f9f4af33767165b61052c 
  src/webui/master/static/ui-bootstrap-tpls-0.4.0.js PRE-CREATION 
  src/webui/master/static/ui-bootstrap-tpls-0.4.0.min.js PRE-CREATION 

Diff: https://reviews.apache.org/r/12395/diff/


Testing
-------

Rendered all pages of the site.

Rendered all Popovers and Modals.


Thanks,

Ross Allen


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message