/srv/irclogs.ubuntu.com/2012/11/09/#juju-gui.txt

bacmorning mattuk197211:49
mattuk1972hey11:49
mattuk1972bac: hey11:50
bacmattuk1972: did you ever get a chance to look at those cance/deploy buttons and decide if the latest tweaks are acceptable?11:50
mattuk1972i need to try and grap head of visual11:50
mattuk1972grap even11:50
mattuk1972grab11:50
mattuk1972!11:50
mattuk1972because he's the one we have to get it past11:51
bacmattuk1972: ok, cool.  i've got a card for it in UX review and get asked about it daily... :)11:51
mattuk1972ill make sure i pass it by him11:51
mattuk1972im sorry11:51
mattuk1972he's hard to pin down11:51
bacno prob.  gary bugs me.  i bug you.11:51
mattuk1972:-)11:51
mattuk1972was just going through the main charm store page and theres a bit of feedback (as usual)11:52
frankbanbac: if you have time, could you please take a look at https://codereview.appspot.com/6827066? mostly CSS stuff.12:03
frankbanmattuk1972: hi, I have a branch in review re charm confing panel, needing UX review. Do you want me to send screenshot, or do you prefer to wait for the branch to be landed and then review in staging?12:05
mattuk1972frankban: screenshot should be fine12:27
frankbanmattuk1972: cool, thanks12:27
bacfrankban: yes12:44
frankbanbac: thanks12:45
bacfrankban: in your MP description you say you followed the wireframe.  is that accurate or did you use the visual design document?12:47
baccharm-store-configure-layoutguide.pdf12:47
frankbanbac: both12:48
frankbanI used the png for measurements and the pdf for colors, sizes, etc...12:50
bacfrankban: cool13:02
mattuk1972bac: store_front_review.pdf uploaded in char store folder of visual design13:06
bacmattuk1972: ok13:06
gary_postermattuk1972, hi.  will you be around for today's retrospective call (right after the kanban call)?  we have a topic to discuss best ways to coordinate around designs and assets. If you can be there I'll put that first (after the kanban bits) so you can bow out after13:08
mattuk1972sure13:09
gary_posterty13:09
bacfrankban: nice function documentation!13:14
mattuk1972frankban: is that 16px text in the text fields? e.g. cassandra13:18
frankbanmattuk1972: yes13:20
mattuk1972I think it should be 14. its just too huuuuuge? the text seems to line oddly in the vertical?  sometimes its really close to the top and sometimes to the bottom?13:22
mattuk1972frankban: also the cancel and confirm buttons should be the same size really13:23
frankbanmattuk1972: for 14px, no problem, for the buttons, you mean the width?13:28
mattuk1972frankban: ya13:28
frankbanok, noted13:29
mattuk1972frankban: also everything should left align to the same place - at the moment the title and buttons and content all align diferently13:30
mattuk1972frankban: everything should be 10px in from the single grey pixel line div.13:31
frankbanmattuk1972: ok, everything with 10px left padding. ah! but in the charm detail panel the padding is 12px everywhere. mattuk1972: I think we should go one way or the other13:34
mattuk1972frankban: its 12 pixel from my original design where the measurement was taken from the edge of the charm panel - it seems like the measurement here is being taken from the grey single pixel line. also in my original design this was a two pixel line anyway13:47
mattuk1972frankban: so that makes it 10 px13:48
frankbanmattuk1972: ok13:49
mattuk1972frankban: sorry to be a nitpicker13:49
bachi, i'm seeing this conversation late.  the charmstore_assets_guide.pdf has 11px on the left, mattuk1972 and frankban13:50
frankbanmattuk1972: :-) np13:50
bacis that wrong or not taking a border into effect?13:51
bacwould be nice to see it standardized and abstracted into a LESS variable13:51
mattuk1972i was just trying to give the measurement from how i thought it was being taken13:52
frankbanbac: less variable, agreed13:52
mattuk1972in my original layout guide i gave it as 12px not taking border into effect13:52
mattuk1972sorry for any confusion13:52
frankbanmattuk1972: so, I will change it to 10px border excluded13:54
mattuk1972there is a slight issue in that the two px border isnt there - i was trying to give the panel an inset look that i think is lost now13:55
gary_postertveronezi, I just gave a follow-up review.  Things are looking good to me.  I only have one non-trivial suggestion, and hopefully it is pretty easy one way or the other.13:58
tveronezigary_poster: g+?13:59
gary_postersure13:59
frankbanmattuk1972: we should make another card for the 2 pixel border IMHO. I am going to do that if you agree.14:02
mattuk1972frankban: sure14:02
frankbanbac: I am fixing the branch you are reviewing introducing the padding-left variable for the charm panel14:03
frankbanmattuk1972: cool14:03
bacfrankban: ok14:03
bacfrankban: review sent14:03
bacnothing surprising in it14:04
frankbanbac: thanks. re: the config file... in my browser it is part of the vcollapsible, actually collapses and the chevron works well14:06
frankban:-/14:06
bacfrankban: i'm using chrome on quantal14:07
frankbanbac: me too14:07
frankbanbac: cache?14:07
bacfrankban: so you collapse 'server settings' and the button goes away?14:08
bacperhaps i described it poorly14:08
frankbanbac: yes14:08
bacfrankban: i will try clearing cache14:08
mattuk1972bac: frankban: garyposter: the use config file ux has never been properly looked at imo. Jovan is going to take a look at it to figure it out correctly.14:16
bacmattuk1972: that would be great14:17
mattuk1972shall i ask him to make a card?14:17
benjibcsaller: do you have any pointers on how the d3 world does testing?14:25
benjiI keep finding things about blood tests for vitamin levels.14:26
gary_posterheh14:27
bcsallerbenji: ha, I'm not sure what to tell you, they have unit tests in the code base that are pretty good, but our more specific tests don't really translate to how those work14:27
bcsallerfor things like the env view I think we end up with lots of calls to simulate like we have now. At the unit test level I've tried to make the framework refactor a little more testable14:28
bcsaller(and tested)14:28
bachi mattuk1972, i'm looking at store_front_review and have some questions.14:31
mattuk1972yup14:31
mattuk1972want to do a call?14:32
gary_posterfrankban, you still need a second review for charm-config-panel, yeah?  starting it, so stop me if not necessary :-)14:32
bacthe cell height comment is confusing.14:32
frankbanbac: pushed fixes per UX review, and also LESS variable to set all the charm panel views to 10px left padding. still seeing that ugly config file behavior?14:32
bac70px when a one line description?  what if there are multi-lines?  i'm not sure how that could even be handled in css14:32
frankbangary_poster: yes I do, thanks!14:32
gary_postercool, wlecome14:32
frankbangary_poster: and just pushed fixes per UX review14:32
gary_postergot it, pulled14:33
mattuk1972bad, a fixed cell height no matter what text - not nice to resize14:33
mattuk1972the cell14:33
mattuk1972bac not bad14:34
mattuk1972...14:34
bacbad bac!14:34
mattuk1972lol14:34
gary_poster:-)14:34
bacand what if there is a long description mattuk1972?  we have no control over it as it is provided by charm authors14:35
mattuk1972two line and then … i guess14:35
mattuk1972much like my email prog14:35
bacwe can do a *minimum* of 70px, which will cover the one line case but i think it'd be nicer to see it grow than have text truncated.14:36
benjiwe might, almost, be at the point where most browsers can do ellipsis correctly14:36
baclet me play with it14:36
bacbenji: just automatically?  or does something need to be specified?14:37
benjibac: it is automatic.  I know that recent FF and Chrome support it, I would hope IE does by now too14:37
benjibac: text-overflow: ellipsis;14:37
bacfrankban: i'm seeing the proper collapsing now.  the animation is crazy slow14:38
mattuk1972cell grow bad - a fixed structure helps to scan tabular info14:38
bacbenji: cool14:38
gary_postercollapsing: weird, quite fast for me14:39
bacgary_poster: ok.  must be my odd environment14:40
bacchrome over x14:40
gary_posterbac, that doesn't mean we should ignore it14:40
gary_posteroh14:40
gary_posterwell maybe it does then :-P14:40
frankbanbac: fast for me too. maybe you are looking at the position, so you can not catch the speed, it's quantal mechanics 14:40
benjibac: yep, it looks like all major browsers support ellipsis correctly now.  I have waited so long for this day.14:41
benjiI need to get out more.14:41
gary_poster:-)14:41
mattuk1972hehe14:41
bacgary_poster: i think i am going to work on monday and take it as a swap day later.14:42
gary_posterbac, cool.  (me too)14:42
gary_posterday after Thanksgiving is just too convenient14:42
benjigary_poster: how many days do we need to reserve for the Christmas shutdown?  I seem to recall it being four.14:43
* gary_poster looks14:43
* benji wonders where gary_poster looks for these things.14:43
bacbenji: it is only the canonical calendar14:48
gary_posterbenji, I search in myriad wonderful places until I find what I want :-P14:48
gary_posterbac, to what do you refer?  I was just very proud to have found https://docs.google.com/a/canonical.com/file/d/0ByrI0-OYdkUEMzA4OTk2MWYtNmVhNC00ZTAxLWI1YTItNTg5ODQ1NWRlNTcx/edit?hl=en_US14:48
gary_posterwhich shows what I suspected:14:48
gary_posterwe get Friday Dec 28 off generally14:49
gary_posterso in the US we need to reserve three days for Christmas holidas, benji14:49
gary_posteroh wait, no, 414:49
gary_poster24, 26, 27, 3114:49
bacgary_poster: launchpad team google cal shows the span.  but doesn't show the 28th as free14:50
bacit also doesn't show the 24th as being closed14:51
gary_posterright, that was the important part that I vaguely remembered14:51
bacso yours wins!14:51
gary_poster:-)14:51
mattuk1972bac: the visual design manager he's says the buttons are good to go : -) he said marc s may pick up on the curve but as long as we have a good reason why they are this way it should be good14:56
gary_posteryay!14:56
mattuk1972woop14:56
gary_poster:-)14:56
bacexcellent14:56
benjiSanity prevails.14:57
gary_posterfrankban, approved as is.  looks great14:58
frankbangary_poster: cool thanks14:59
gary_posterteknico, no rush, but I'm in hangout from calendar15:01
teknicogary_poster, getting there15:02
gary_postercool15:02
tveronezigary_poster: quick g+?15:15
gary_postertveronezi, on another call sorry :-015:16
gary_poster:-) I mean15:16
tveroneziok.15:16
tveronezigary_poster: I just want to discuss the "bootstrap" thing. Just want to let you know that if we use that, the "aliases" (the "use" key in "modules.js") wont work. I will need to specify the requirements one by one. I dont know what to do now.15:19
tveronezigary_poster: ping me when you have time.15:19
gary_postertveronezi, ok, will try to do it asap15:19
gary_posterbac bcsaller benji frankban jovan2 Makyo mattuk1972 teknico tveronezi call in 2 in juju-ui15:58
gary_postertveronezi, sorry; I'm there now if you want to try and squeeze something in; otherwise maybe afterwards would be good15:59
=== mattuk1972_ is now known as mattuk1972
gary_posterMakyo starting16:01
bacjovan2: the first 'Deploy' button could be re-labeled 'Configure' as it leads to the configure panel.16:21
=== gary_poster|away is now known as gary_poster
=== gary_poster is now known as gary_poster|away
=== gary_poster|away is now known as gary_poster
Makyobcsaller, can I request you as a reviewer on the firefox thing, whenver's convenient?  I'm messing with drawing relations, figure you should be in on that,.17:08
bcsallerMakyo: sounds good, I'll look at it after some food17:08
MakyoCool, thanks.17:09
=== gary_poster|away is now known as gary_poster
gary_posterOK, I might be back...17:17
tveronezigary_poster: g+?17:52
gary_postertveronezi, ok if you don't mind occasional distractions for me17:53
bacbenji: wrt to wrapping and ellipsis, i added text-overflow: ellipsis as you suggested18:52
benjidid we settle on tags or multiple review requests to keep up with who was reviewing what?18:52
bacbut it isn't overflowing and ellipsising.  it is just spilling into the next <li>18:52
benjibac: what browser?18:53
bacbenji: tags separated by vertical space18:53
benjiheh18:53
bacchrome18:53
bacbenji: and hunter suggested i remove the fan blades to see if it speeds up.18:53
bacif it does, then they will replace the blades18:53
benjibac: oh, you need "white-space: nowrap" too18:53
bacrinse, repeat18:54
benjithat's... odd18:54
benjido they occasionally make blades that are way too heavy?18:55
bacthe way i figure it, a bad motor could speed up if you remove perfectly good blades18:55
bacso it seems to be a non-conclusive test18:56
bacif the motor does not speed up then it is a bad motor18:56
benjiMakyo: I was going to review your branch, but I can't find it.  ("Cannot remove subordinate relations")19:24
bacbenji: do you have a sec for a call?19:30
benjibac: sure19:30
Makyobenji, lp:~makyo/juju-gui/remove-sub-rels19:47
benjiMakyo: thanks19:47
benjiMakyo: is there a reitveld review or MP?19:51
MakyoAh, sorry. https://codereview.appspot.com/6782063/19:52
bacbenji: http://danbeam.org/ellipsis/yui/19:53
benjibac: cool19:53
* bac walkies. bbiab.20:12
gary_postertveronezi, your branch has been fully blessed :-)20:49
tveronezitak! :)20:49
tveronezithanks! I mean! :)20:49
gary_posterI thought you were turning Danish on us!20:49
gary_posterMakyo, trying to quickly give you a review before a call in 11 minutes so you can land replace-service-module...20:49
Makyogary_poster, Thanks:)20:54
gary_poster:-)20:54
gary_posterWow, Makyo, FF is now completely usable for me on both new desktop and 3 year old laptop: congrats!20:56
gary_posterChrome still faster20:56
gary_posterbut the difference in FF is huge20:56
gary_posterapproved Makyo, witgh trivial comments21:02
Makyogary_poster, Thanks.  I know there's still room to improve FF, but that may come with other branches.  Will take care of comments, etc.21:04
gary_postercool thx21:04
* tveronezi brb21:22
* tveronezi back21:32

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!