-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21247/
-----------------------------------------------------------
(Updated May 14, 2014, 1:30 a.m.)
Review request for Aurora, Suman Karumuri and Bill Farner.
Changes
-------
The config bar is now always on, Suman got a +1 on his viewpoint from a design
resource in Seattle so I'll considered myself overruled.
All of the interaction and event handlers for my directive have been moved
inside the directive itself. The controller now only has code to munge the data
into the correct format. Now any controller which wants to reuse this directive
just needs to get the data into the same format - at which point we should move
that logic into a service.
Bugs: AURORA-378
https://issues.apache.org/jira/browse/AURORA-378
Repository: aurora
Description
-------
Replace the button to show/hide configs with a bar when there are multiple.
Tidied up the display of the configuration data.
Diffs (updated)
-----
src/main/java/org/apache/aurora/scheduler/http/ServletModule.java
983101277ffbd1c4017b29f4c86e61315f1bcc78
src/main/resources/org/apache/aurora/scheduler/http/ui/configSummary.html
PRE-CREATION
src/main/resources/org/apache/aurora/scheduler/http/ui/css/app.css
dc515ac818a9af36522bb07a125cd92ff9230fa2
src/main/resources/org/apache/aurora/scheduler/http/ui/groupSummary.html
PRE-CREATION
src/main/resources/org/apache/aurora/scheduler/http/ui/job.html
cd78c20bee7891a6cdfd19943a6f7e8a9dce33df
src/main/resources/org/apache/aurora/scheduler/http/ui/js/controllers.js
7c80bd769b7e80bef0822846166959925b1bf7f8
src/main/resources/org/apache/aurora/scheduler/http/ui/js/directives.js
a79276d2486fc1122a8b084ea0614cdae759f88c
Diff: https://reviews.apache.org/r/21247/diff/
Testing
-------
File Attachments
----------------
Job page with only one config group
https://reviews.apache.org/media/uploaded/files/2014/05/08/908570f2-bef2-4370-bf08-1b5c7400c3c6__Screen_Shot_2014-05-08_at_4.26.02_PM.png
Showing config for only one config group
https://reviews.apache.org/media/uploaded/files/2014/05/08/1424f2c3-cf66-4ce4-a595-dc3f7647d6d2__Screen_Shot_2014-05-08_at_4.26.12_PM.png
Job page with multiple config groups
https://reviews.apache.org/media/uploaded/files/2014/05/08/bc9c1616-3c71-4bcc-ab40-fefc56d5e19a__Screen_Shot_2014-05-08_at_4.25.21_PM.png
Viewing one of the groups from config bar
https://reviews.apache.org/media/uploaded/files/2014/05/08/6b8e6077-c8c8-48c9-94e9-732e782204b6__Screen_Shot_2014-05-08_at_4.25.36_PM.png
Viewing multiple configs at the same time
https://reviews.apache.org/media/uploaded/files/2014/05/08/b7b2cba1-034b-43e6-9eaa-3ba1a78f87f5__Screen_Shot_2014-05-08_at_4.25.42_PM.png
screenshot of really small group
https://reviews.apache.org/media/uploaded/files/2014/05/12/2d7999ac-ba1f-45b0-bf9e-99f9c7a10009__Screen_Shot_2014-05-12_at_1.27.59_PM.png
Thanks,
David McLaughlin
