On Jan 17, 2014, at 9:34 PM, Brian Federle <brian.fede...@citrix.com> wrote:
> OK, I'm dropping this subject after this, but one more thing -- the new UI > has been up for months now. There was plenty of time to make feedback and > improve the UI. I don't appreciate the fact that these issues are being > mentioned so late into release, when there isn't much we can do about it. > Also, we are working on other deliverables for future releases, so the fact > that our bug count is low for 4.3 doesn't mean we aren't doing any work :) I understand the 'lateness' of my testing. I did not see a [MERGE] request for this important new feature and searching on the dev mailing list I did not find one, maybe I missed it. The only thing I found was the wiki page: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Update+UI+visual+appearance and the bug https://issues.apache.org/jira/browse/CLOUDSTACK-4748 The bug mentions the 'ui-restyle' branch, the version being affected is not specified. But the Basic reskin has been solved 5158 and applied to 4.3: https://issues.apache.org/jira/browse/CLOUDSTACK-5158 (too bad there is no commit message or merge request link in the bug, that would be nice). 5158 seems to be a work around using the existing CSS due to the complexity of the refactor. My point is that if it's still a work in progress why pushing it to 4.3 ? You can keep on working in the feature branch and once it's ready call a 'MERGE' request to give a chance to everyone to give feedback and file bugs. I hope to see a 'MERGE' request for what will be in 4.4. -Sebastin > > -----Original Message----- > From: Sebastien Goasguen [mailto:run...@gmail.com] > Sent: Friday, January 17, 2014 11:53 AM > To: dev@cloudstack.apache.org; Brian Federle > Cc: Jessica Wang; Sonny Chhen > Subject: Re: UI blocker > > > On Jan 17, 2014, at 1:41 PM, Brian Federle <brian.fede...@citrix.com> wrote: > >> Sorry, but cosmetic issues are not blockers :) It would only be a blockers >> if the functionality were NOT usable in any shape or form, for both testing >> and end use. >> > > I disagree, cosmetics i.e look and feel of a new major revision of a software > are as important as critical backend features. Because users will look at it, > managers, execs...etc, it will be displayed in demos, printed on booths > etc....anything that does not look top notch reflects badly on the project > and the software. > > >> i.e. -- if the fields were completely missing or garbled in some way, I >> would agree that we can classify the bug as blocking release. Looking >> 'ugly', no -- I implemented the scrollbars because most times API keys, etc. >> are usually very long strings, causing overflow on some browsers and in most >> usability cases people are mainly needing to just copy/paste the values into >> other fields. It's not the most elegant solution I understand, but this is >> also a small facet of the whole UI experience. >> > > Attention to details is important. Personally the new UI is distracting to > me, the variation in fonts/font sizes is distracting instead of putting > emphasis and focus. There are other things, like 'ovm' still showing up in > the add cluster drop down when OVM is not supported in CloudStack, but you > are right it's a detail who cares. > >> I am going to ask everyone to PLEASE respect this when determining priority >> on UI bugs -- I know things like this are somewhat ugly, but there are only >> a small handful of full-time UI developers working on CloudStack and it >> makes our lives harder if everyone is marking trivial stuff as show-stoppers. >> > > This is great, so it can look like crap, as long as it works we release on > the first iteration of a major revision. In any case you can remove this as a > blocker, it seems I am in minority here, at least I got you to respond on the > list and share your thoughts on the subject. As a parting thought though, I > don't think that 'everyone is making trivial stuff as show-stoppers', I did > not hear many UI blocker bugs lately > >> At any rate, if you still think this is a blocker, you can feel free to look >> into it yourself, in which case write a review request and I'll take a look >> at it. > >> >> -Brian >> >> -----Original Message----- >> From: Sebastien Goasguen [mailto:run...@gmail.com] >> Sent: Thursday, January 16, 2014 11:56 AM >> To: Brian Federle >> Cc: dev@cloudstack.apache.org; Jessica Wang >> Subject: Re: UI blocker >> >> >> On Jan 16, 2014, at 2:48 PM, Brian Federle <brian.fede...@citrix.com> wrote: >> >>>>> Did you happen to check the View Users page ? the api/secret keys display >>>>> is being truncated. >>> >>> Yes, though to fix that I'll have to rework a bit of the detail view layout >>> to allow more flexible sizing for long strings. File that as a separate bug >>> for the next release and I'll take a look at it. >> >> No, I will keep that as a blocker. it's worse than the previous UI, feel >> free to comment in the bug and grab it. >> >>> >>> -----Original Message----- >>> From: Sebastien Goasguen [mailto:run...@gmail.com] >>> Sent: Thursday, January 16, 2014 11:17 AM >>> To: Brian Federle >>> Cc: dev@cloudstack.apache.org; Jessica Wang >>> Subject: Re: UI blocker >>> >>> >>> On Jan 16, 2014, at 1:39 PM, Brian Federle <brian.fede...@citrix.com> wrote: >>> >>>> Don't see any issues in the screenshot you posted. The variation in font >>>> size and weight, color, etc. are used to show contrast between content and >>>> titles. Sorry but I do not see this as a blocker issue :) Though feel free >>>> to tweak the fonts yourself. >>>> >>> >>> It looks strange to me, maybe that's just me. I see too many variations and >>> it's disturbing. >>> >>> Did you happen to check the View Users page ? the api/secret keys display >>> is being truncated. >>> >>>> -Brian >>>> >>>> -----Original Message----- >>>> From: sebgoa [mailto:run...@gmail.com] >>>> Sent: Thursday, January 16, 2014 1:43 AM >>>> To: dev@cloudstack.apache.org >>>> Cc: Jessica Wang; Brian Federle >>>> Subject: UI blocker >>>> >>>> Hi, after testing 4.3 I entered a blocker due to UI issues: >>>> https://issues.apache.org/jira/browse/CLOUDSTACK-5882 >>>> >>>> It would be good to discuss, >>>> >>>> thanks, >>>> >>>> -Sebastien >>> >> >
