----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71059/#review216539 -----------------------------------------------------------
Ship it! Very nice!! src/webui/app/roles/roles.html Lines 31 (patched) <https://reviews.apache.org/r/71059/#comment303750> Plural? ditto below src/webui/app/roles/roles.html Lines 34 (patched) <https://reviews.apache.org/r/71059/#comment303751> Plural? Ditto below - Meng Zhu On July 11, 2019, 10:07 a.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71059/ > ----------------------------------------------------------- > > (Updated July 11, 2019, 10:07 a.m.) > > > Review request for mesos, Armand Grillet, Andrei Sekretenko, and Meng Zhu. > > > Bugs: MESOS-8789 > https://issues.apache.org/jira/browse/MESOS-8789 > > > Repository: mesos > > > Description > ------- > > This adds resource breakdowns on a per role basis to the roles table > in the webui, each column except for the role name is selectable to > allow the user to tune the view for their needs. > > > Diffs > ----- > > src/webui/app/controllers.js 42a9fb657e98bbd57265f2c9cf8e4d1013d3f271 > src/webui/app/roles/roles.html 45e2a332e62e072e97fc97ab3945ff65ad942ddf > > > Diff: https://reviews.apache.org/r/71059/diff/1/ > > > Testing > ------- > > See attached gif. > > > File Attachments > ---------------- > > demo > > https://reviews.apache.org/media/uploaded/files/2019/07/11/44987278-1415-4e55-a36f-2fb299d930e0__roles-table-selectable-columns2.gif > > > Thanks, > > Benjamin Mahler > >
