News for hackers. Stuff that matters.
Re:CodeFlow: Improving the Code Review Process at Microsoft ... by m0zg on Wednesday February 06, 2019 @11:20PM (#19102972)
Thank you for putting it together! I switched folks over and we've been paying customers for well over a year now. I can overlook the UI deficiencies when I have proper review workflow, but the problem is that people who don't know the Critique workflow might be put off by the UI. Personally, I encourage them to give Reviewable a good try anyway. Anything larger than a few dozen lines and/or needing multiple rounds of feedback is just untenable in plain GitHub. I can say with certainty, our code would be worse if it wasn't for Reviewable.
Re:CodeFlow: Improving the Code Review Process at Microsoft ... by piotrkaminski on Wednesday February 06, 2019 @11:41PM (#19103044)
Thanks again, I might have to quote you on the home page. :) And yes, the UI is often the sticking point for adoption. I don't know if it's just the style / toolkit (an old version of Semantic UI) and switching to something like Material Design would take care of the problem, or if it's a deeper design / UX issue. A good onboarding flow would likely help as well but I haven't figure out what that should look like yet either. More feedback and advice is always appreciated -- and if you happen to be (or know) a UX expert who knows how to fix it I do have some budget I could throw that way. :)
Re:CodeFlow: Improving the Code Review Process at Microsoft ... by osel on Thursday February 07, 2019 @02:58AM (#19103757)
From my brief play with your demo it's a bit of both.

The toolkit is very small and lacks good definition of elements (both visually in that it lacks contrast, and practically in that I have no idea what is clickable or editable).

The page structure isn't that clear either, where are the column headings? And on my widescreen browser window the isn't a limit on horizontal stretch - I have to play with the browser to find a good width. Also, I can expand code, but how do I shrink/hide it?

Re:CodeFlow: Improving the Code Review Process at Microsoft ... by piotrkaminski on Thursday February 07, 2019 @03:03PM (#19108039)
Thanks! Sounds like switching toolkits should address many of these issues.

And no, you can't shrink/hide code, other than by re-diffing. What can I say -- it's surprisingly hard to implement and didn't seem worth it.