[18:28:19] I'm having a very "who moved my cheese!" reaction to polygerrit -- https://phabricator.wikimedia.org/M242 [18:30:09] bd808 something like https://docs.google.com/presentation/d/17q-ygGioZi_5DITLyELa8oaOr22e15AHy8cq6XTZ0nY/edit#slide=id.g27f16618ec_0_139 ? [18:30:30] or would you be able to write a task here https://phabricator.wikimedia.org/project/view/330/ (feedback for polygerrit) [18:30:35] and we can forward it upstream. [18:32:52] It's not quite that blue, yet :) [18:33:00] yep :) [18:33:29] no_justification though it's begining to be. [18:33:36] im still waiting for the new header desgn :) [19:01:47] paladox: I'm probably always going to be a bit crabby about UX. I do "weird" things like run my web browser at a 900 to 1100 pixel width rather than full screen (or even half screen) that designers are likely to optimize for. [19:02:15] I have a lot ot greasemonkey scripts to reduce whitespace and reorder things in tools I use all the time like gerrit, phabricator, and gmail [19:03:35] floaty breezy UI is great for a site where I'm being entertained, but in a working tool I want high density displays that show me jsut what I need to see and pack it into the above the fold view [19:04:50] bd808 heh, polygerrit are looking for feedback, on the ui [19:04:56] dosen't matter how you use it :) [19:05:05] the more feedback, the more better polygerrit comes [19:05:15] (also helps out there ui desgners behind the scenes) [19:05:53] bd808 does this https://gerrit-new.wmflabs.org/r/?polygerrit=1 help [19:06:08] (it's running from the latest gerrit branch (master)) [19:09:19] bd808 upstream are really listening, it's not like how it was before. They have a dedicated team on this ui with ui desgners behind the scene with a person who speak on behave of users in the gerrit ui meetings. [19:18:54] paladox: my initial problem with the UI on gerrit-new is that the commit message and vertical list of metadata is all I see above the fold in my browser. I have to scroll to see anything useful about the change. [19:19:29] bd808 ah. yep. I think they have a big redesgn comming. Per https://docs.google.com/presentation/d/17q-ygGioZi_5DITLyELa8oaOr22e15AHy8cq6XTZ0nY/edit#slide=id.g27f16618ec_0_139 [19:20:01] bd808 i guess you have your browser half way? [19:20:06] so the mobile css is kicking in [19:20:46] 1024 internal view width [19:21:10] if "mobile" is 1024px... wow [19:21:51] bd808 it's 50em [19:22:08] using the meta tag line to detect. [19:22:12] it does seem that they have a breakpoint around 1220 that makes it better (less stacked on the left info column) [19:22:33] https://github.com/GerritCodeReview/gerrit/blob/master/resources/com/google/gerrit/httpd/raw/PolyGerritIndexHtml.soy#L30 [20:16:17] It's so much easier to debug polygerrit. As well it uses the element name (which is the same file name). [22:55:10] bd808 the new ui is now built in pure javascript + html. Non of the java stuff. It uses the rest api to get it's data. it's index.html page is generated by soy, to get the base url that we set in a config. [22:55:33] bd808 it uses polymer which youtube uses [22:55:53] (you can tell, as in the develeper tools for viewing the html source, you see yt-element-name