----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25476/#review52718 -----------------------------------------------------------
ui/scripts/ui/widgets/listView.js <https://reviews.apache.org/r/25476/#comment91697> Should n't this be declared as var, also this is not used or referenced in the function definition. - Rohit Yadav On Sept. 9, 2014, 1:47 p.m., Mihaela Stoica wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25476/ > ----------------------------------------------------------- > > (Updated Sept. 9, 2014, 1:47 p.m.) > > > Review request for cloudstack, Brian Federle and Jessica Wang. > > > Repository: cloudstack-git > > > Description > ------- > > - Preserve the advanced search parameters, so that when the advanced search > box is shown again, > it is populated with the values selected/entered previously, unless they have > navigated away from the search results page, > or applied any additional filters/search parameters. > > > Diffs > ----- > > ui/scripts/ui/widgets/listView.js 22182c1 > > Diff: https://reviews.apache.org/r/25476/diff/ > > > Testing > ------- > > > Thanks, > > Mihaela Stoica > >