> On July 12, 2013, 10:27 p.m., Ben Mahler wrote:
> > Hey Ross, I committed this just before you last updated it. Looks like it 
> > was just a rebase so it's ok, but in the future can you use the 'Depends 
> > On' field or mention the related reviews? It would help me commit in the 
> > right order :)

Aha, sorry about that. I will watch that next time.


- Ross


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


On July 12, 2013, 10:18 p.m., Ross Allen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12395/
> -----------------------------------------------------------
> 
> (Updated July 12, 2013, 10:18 p.m.)
> 
> 
> Review request for mesos.
> 
> 
> 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
> -----
> 
>   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
> 
>

Reply via email to