That is a good idea too, I agree. Also keep in mind that it will require a fair bit of UI testing too, since this would affect numerous dialog boxes and forms throughout the UI.
-Brian On 6/25/12 1:04 PM, "Pranav Saxena" <pranav.sax...@citrix.com> wrote: >Hi Olga , > >I reviewed your code for the following bug which you raised - >http://bugs.cloudstack.org/browse/CS-15292 . This looks like a more >modular improvement in displaying the OS types as split categories. Your >patch looks good however since the backend API command hasn't been >implemented for listing the specific OS types , you'll have to probably >wait a bit for this patch of yours to be merged ( Where you have written >"api command" explicitly in your patch ). > >The idea definitely looks good to me. > >@Brian/Jessica - what are your suggestions for the same ? > >Regards, >Pranav > > >-----Original Message----- >From: Olga Smola [mailto:olya.sm...@gmail.com] >Sent: Wednesday, June 20, 2012 8:56 PM >To: cloudstack-dev@incubator.apache.org >Subject: UI tickets: Review request > >Hi, > >Could you please review my patches for the following tickets: > >http://bugs.cloudstack.org/browse/CS-15026 >http://bugs.cloudstack.org/browse/CS-15292 >http://bugs.cloudstack.org/browse/CS-14110 >http://bugs.cloudstack.org/browse/CS-15290 >http://bugs.cloudstack.org/browse/CS-15286 >http://bugs.cloudstack.org/browse/CS-14047 >http://bugs.cloudstack.org/browse/CS-15287 - Please take a look at the >ticket last comment >http://bugs.cloudstack.org/browse/CS-15315 >http://bugs.cloudstack.org/browse/CS-4460 - Please take a look at the >latest ticket comment. There is a problem, already discussed with Jessica >Wang, but not completed, still need feedback, maybe some ideas how to >correct. If you have ideas, write to me and I'll forward already >discussed with Jessica. > >Thanks, >Olga
