/srv/irclogs.ubuntu.com/2014/11/24/#juju-gui.txt

=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
hatchuiteam I still need two qa's and reviews https://github.com/juju/juju-gui/pull/66415:29
kadams54hatch: taking a look15:29
hatchthanks15:29
rick_h_Makyo: can you look at ^ as well please?15:29
hatchit needs a real env qa15:29
MakyoSure15:37
hatchthanks guys15:41
bachatch: you going to land a juju-gui branch soon? let me know when you do and :shipit:15:49
hatchbac: the doc one wasn't good enough?15:49
hatchor do we need another test?15:49
bachatch: another, i didn't have the site setup for comingsoon to get built properly15:50
hatchohh ok15:50
hatchone is getting qa's so hopefully that'll pass ok and it can be landed15:51
kadams54Makyo: are you QAing hatch's PR in a real env?16:21
MakyoYeah16:21
kadams54I gave it a whirl but ran into problems. Trying to decide if I should try to wade through them or throw my hands up and just review.16:22
hatchkadams54: you should really just work in Ubuntu :)16:30
kadams54Some day16:31
hatchI made a small change to my tests and now they hang16:34
hatch:/16:34
hatchiphone users - can iphones auto sync photos to icloud?16:38
teslanickYes16:39
hatchmom has an old iphone which she says no longer syncs to icloud16:40
hatchwanted to make sure it ever did ;)16:40
teslanickPretty sure all my photos sync to icloud. Actually, it's all the images I put in my photo library, which is a pain. There are a few meme pictures in my "photo album" now. :|16:41
hatchhaha16:41
hatchkadams54: did you push your branch? I dont' see it16:43
kadams54hatch: https://github.com/juju/juju-gui/pull/66616:49
kadams54PR#666 - ominous.16:49
teslanickHALLOWEEN WAS WEEKS AGO16:50
hatchwell 666 is only ominous if you believe in the most recent religions and not the ones which the new ones were based on16:50
hatch(old roomie was a religious studies major) 16:50
hatch:P16:50
* hatch <----- buzkillington16:51
teslanickNote that the real "number of the beast" or whatever is 616. The current "666" is a translation error.16:52
hatchoh interesting16:52
* hatch gets 616 shirts made up for the emo kids16:52
jrwren616 is a nearby areacode. grand rapids, I think.16:57
hatchare they all deamons?16:57
teslanickwell there you go.16:57
hatchhorns, red16:57
hatchyou know the type16:57
kadams54hatch, Makyo: crap, sorry, just realized I pushed the wrong branch for that PR. Hold off on QA for a few minutes while I straighten things out.17:01
hatchkadams54: np I'm still reviewing huws 17:02
hatchyours got bumped ;)17:02
rick_h_hatch: you see http://www.fse.guru/2-years-with-angular ?17:08
hatchI have not17:08
hatchI will read it later though17:08
=== teslanick is now known as |conj
=== |conj is now known as |conj_teslanick_
=== |conj_teslanick_ is now known as cat_calories|tes
=== cat_calories|tes is now known as teslanick|eatfoo
hatchkadams54: you pulled that PR down?17:23
hatchoh I thought you were just going to update it17:24
hatch:)17:24
hatchnm, ping when ready17:24
kadams54hatch: Yeah, https://github.com/juju/juju-gui/pull/66717:24
hatchcool 17:24
=== teslanick|eatfoo is now known as teslanick
hatchkadams54: so the status bar seems a few pixels too high in it's little box 17:26
kadams54Makyo: ^ New PR ready for QA.17:26
hatchdoes it look like that to you too?17:26
hatchmaybe it's just an illusion :)17:26
kadams54hatch: No, not particularly.17:27
kadams54FWIW, I didn't change any CSS for this - just shifted stuff around in the HTML.17:27
kadams54So visually the status bar is the same.17:27
hatchThat add units header sure looks bad now haha17:27
hatchkadams54: so the scale up container element...it was missing before?17:29
hatchwas it just working by chance?17:29
kadams54No, wasn't working by chance.17:29
kadams54The container element was a new div rather than an existing one.17:30
kadams54serviceOverview appended that div into its container, which guaranteed that it would always come first.17:30
kadams54Since I needed the status bar div to come first, I had to switch to having a pre-existing placeholder div which would be passed in as the container to scaleUp.17:30
kadams54hatch: Your gripes about the Add Unit header are worth mentioning to luca. I just double-checked the mocks attached to the bug and there's no header there. The status bar is almost merged with the scaleUp UI.17:31
kadams54hatch, luca: here's the mock: http://cl.ly/image/1C1c0i1k0Q2e17:32
kadams54And here's the current impl: http://cl.ly/image/2U3D3X0y0N3r17:32
kadams54I don't think it would be too hard to pull the header out as part of this branch.17:32
hatchwell you now have 2 +1's so might as well just land it and we can follow-up late17:33
hatchr 17:33
lucakadams54: well, it’s positioned better at least :)17:33
hatchluca:  you'll have to take a look once it lands, the header kind of looks out of placed17:34
hatchit doesn't really look like it 'belongs' to anything17:34
lucakadams54: hatch Makyo we did come up with a new way of displaying this in Nuremberg17:35
hatchyeah? 17:35
hatchhave any mocks to look ati?17:35
lucakadams54: hatch Makyo I haven’t wireframed it yet but we made some great improvements17:35
hatchohh ok17:35
kadams54OK17:35
hatchI know it's been a tough problem to solve17:35
lucakadams54: hatch Makyo no…and I remember now that it was actually with Huw17:35
lucai’ll send my notes on it17:35
hatchhuw was in nuremberg?17:36
kadams54luca, hatch: for what it's worth, here's how things would look if I just dropped the <h2>: http://cl.ly/image/470F3H3f1n1d17:36
hatchor are you thinking brussels?17:36
hatchheh name dropping destinations17:36
hatch:P17:36
kadams54I prefer that and it's a simple change, esp. if we're going to rework that shortly.17:36
hatchkadams54: need bigger shot17:37
hatchmore context of the inspector :)17:37
kadams54http://cl.ly/image/293x1t170a2417:37
hatchyeah that's much nicer imho17:38
hatchkadams54: I also see your cache is broken again ;)17:39
kadams54Yes17:39
kadams54Eternally17:39
hatchand your fonts are way more bold17:39
hatchvery odd...17:39
kadams54I wonder if that's just Chrome/Mac17:39
kadams54hatch, Makyo: pushed the dropped header.17:41
lucakadams54: hatch I have to run but I sent the info via peeps17:41
kadams54I gotta run for my doctor appt. but will get this landed and finish my other QA after.17:41
=== kadams54 is now known as kadams54-away
=== teslanick1 is now known as teslanick
hatchrick_h_: I shared a vid on G+ this weekend about some really cool stuff coming in chrome wrt offline/app-like performance19:54
hatcha lot of which would really speed the gui up19:54
rick_h_hatch: yea, I have been meaning to look at it19:54
kadams54rick_h_: you asked me to QA a card in standup, but now I forget which one…19:54
rick_h_kadams54: I think we're ok. I think it was storefront and I looked at it and have questions/etc19:55
kadams54OK. Starting in on the switchboard integration.19:56
teslanickhatch: Is that serviceworker stuff?19:57
rick_h_man, the wind is trying to huff and puff and blow my house down19:59
kadams54rick_h_: yeah, it knocked my recycling over. My porch is a mess of plastic and cardboard right now.19:59
hatchteslanick: yup20:07
hatchuiteam - they are starting on my place so I may or may not get kicked offline :)20:08
teslanickThis is pretty impressive - http://www.youtube.com/watch?v=px-J9Ghvcx420:08
kadams54hatch: geez, is your fiber under perpetual construction?20:08
hatchkadams54: no this time it's new gas meter20:09
hatchand power meter20:09
hatchunless they run into issues20:09
hatchso...yeah20:09
kadams54Like cutting your fiber20:09
kadams54;-)20:09
hatchteslanick: yeah that demo is awesome20:09
hatchkadams54: lol exactly20:09
kadams54teslanick, hatch: pretty sweet20:09
hatchteslanick: serviceworker - what appcache was supposed to be ;)20:09
hatchthere is also some really cool manifest stuff wrt meta tags20:10
hatchso that we don't need to put 100 meta tags at the top of every page20:10
jrwrenwe have 50+mph winds right now. I'm amazed that I have power.20:11
hatchnice!20:11
* hatch grabs kite - heads to Detroit20:11
jrwrenits frame had better be made of something strong.20:12
rick_h_Wind20:12
rick_h_SW 28 mph gusting to 50 mph20:12
hatchbac: I'm about to shippit a branch20:13
hatchjrwren: no frame :)20:13
hatchjrwren: http://www.kittyhawk.com/wp-content/uploads/2012/02/L8G0244-e1328583281953-780x300.jpg20:14
hatchjust a random pic 20:14
hatchit's like an airfoil - it uses the airflow to inflate 20:14
jrwrenhatch: whoa, cool.20:15
teslanickhatch: It's almost like having a real programming language helps you solve problems more effectively20:15
* teslanick glares at CSS.20:16
kadams54jrwren: yeah, it helps to realize that when hatch talks about a kite, he doesn't mean the one we all played with as kids :-)20:17
bachatch: kadams54 beat you to it.  let's see if his updates comingsoon properly20:20
hatchkadams54: jrwren haha no, mine are 5 - 14m of area :)20:21
kadams54bac: ah yes… hatch was too slow :-)20:21
hatchlol20:21
hatchteslanick: haha yeah I've spent many a unable to sleep nights nerding out on how to make a better css :)20:22
hatchso far I've come up empty20:22
teslanickA simple "box model shader" API, written for JS, that uses the "top down" model that React uses.20:23
teslanickSOLVING THE PROBLEM ONCE AND FOR ALL20:23
hatchok well there ARE solutions to the problem - just none that I think are any better20:24
=== kadams54 is now known as kadams54-away
bacuiteam: great, http://comingsoon.jujucharms.com/juju-ui/version.js is updated for kadams54-away recent landing20:50
rick_h_bac: woot ty much!20:51
hatchnice! good work bac20:51
bachatch: your blog post is nice but my install was much simpler as i blew away os x20:53
hatchohh ok20:53
hatchyeah for ppl with new apple hardware you definitely don't want to do that if you want things to work :)20:54
hatchmaybe 16.04?20:54
hatchlol20:54
hatchI don't even know where to start to write drivers for the webcam20:54
hatchand I'd imagine there aren't many new MBP users who run ubuntu who are also driver authors :)20:55
bachatch: i plan to close the lid and just ssh to it20:55
bachatch: don't plan on it being my desktop20:55
hatchI have an oldish unused mac mini that i'd like to turn into my work machine - but I find I leave my desk and work around the house too much20:55
hatchso I'm thinking to think of a workflow that will make it less painful20:56
hatchworking over ssh is too slow for me20:56
rick_h_it depends on the kind of work20:56
rick_h_browser work is a bit harder for sure20:56
hatchyeah - and I don't do any browser work ;)20:56
rick_h_:P20:56
hatchI was thinking some kind of an automatic rsync20:57
rick_h_ugh20:57
hatchso as soon as the machine boots it syncs the home directory20:57
=== kadams54-away is now known as kadams54
kadams54bac: woot!20:58
hatchrick_h_: I was also thinking of running a parallels vm on the mini, and on my laptop so when I leave I can copy it over21:00
hatchbut that's probably worse than rsync lol21:01
hatchffs promises21:10
hatchthat is all21:10
teslanickur doin' it wrogn.21:12
teslanick;)21:12
hatchlol21:13
* jrwren sings promises promises.21:20
teslanickhatch: http://2014.jsconf.eu/speakers/james-coglan-practical-functional-programming-pick-two.html - We disagree about Promises, but I think this is worth a watch even if you're a Promiseskeptic.21:22
rick_h_hatch: just remember, it's all about the implementation :P21:23
hatchlol - hey I think promises are awesome - just not ones which implicitly throw() 21:23
teslanickPromiskeptik. Darn.21:23
hatchI am actually somewhat interested about promise + generator combo21:29
hatchwould be nice for us and large complex imports21:29
hatchs/and/with21:30
teslanickHow do you mean?21:30
hatchlemme find the video21:33
hatchone sec21:33
hatchhttps://www.youtube.com/watch?v=qbKWsbJ76-s21:33
hatchsorry it's long21:33
hatchwell 31min21:34
hatchbut it does a better job outlining what I'm talking about :)21:34
teslanickOh yeah, promises + generators is great. I was mostly curious about the large/complex imports part.21:34
teslanickI wrote some code that implements core.async-style channels and blocks using promises + generators.21:34
hatchohh when people drop a 20+ service with many machines and stuff onto tha canvas21:34
hatchand we need to parse it all out, and deploy it21:34
teslanickAha21:35
teslanickAnd there's a lot of asynchrony there21:35
hatchwell async and sync haha21:35
hatchit's a syncy mess21:35
teslanickHere's what I was fooling with - https://gist.github.com/nhusher/951104421:36
teslanickChannels have three functions: put, take, and close.21:37
=== kadams54 is now known as kadams54-away
hatchinteresting I'll have to take a look later21:38
hatchright now I"m fighting with tests failing when run with the rest of the suite21:38
rick_h_hatch: when the export thing is landed let me know. I think it might be worth doing a release update after that on the gui21:39
teslanickRough. I'll leave you alone21:39
rick_h_hatch: if we can squeeze it in cool, else we'll wait for next release. We're |--| close to our weekly cycle so no problem either way21:39
hatchrick_h_: that's done21:40
hatchand landed21:40
hatchI'm fighting with the charmstore stuff21:40
rick_h_hatch: oh, crap ok 21:40
hatchapparently the tests faill dramatically when I try and remove the icon stuff from v3 because so many of the tests rely on that21:40
rick_h_lol21:45
rick_h_yea, there was a test helper to be used to mock it out21:45
hatchMakyo: I've now removed all of the old iconpath stuff https://github.com/juju/juju-gui/pull/66222:31
hatch33 files changed...oy 22:31
rick_h_hatch: so why does that code go away vs just getting updated?22:45
hatchnot sure I understand the question22:46
hatchwhy did I remove it from charmworld.js?22:46
hatchso that it would fall on its face hard if anyone tried to use it :022:46
rick_h_hatch: so why did you have to touch all those points22:47
hatchbecause those old places use a stubbed version of apiv322:47
hatchso it's now a stubbed version of apiv422:47
hatchdifferent method name22:47
hatchiconpath to getIconPath22:47
rick_h_ok, I see so it didn't go away just needed a lot of rewording22:48
hatchoh no it's gone22:48
hatchI deleted the iconpath method from charmworld.js22:48
hatchall the tests are converted to use the new method in the new api22:49
hatchand everything in the code now uses the apiv4 method22:49
hatchso any icons being fetched are being fetched from v422:49
hatchwhich also means this branch cannot land until the next v4 api update22:50
rick_h_working on getting it through CI now22:50
rick_h_darn mojo CI got me with a failure for some reason22:50
hatchyeah no problem - I don't want to land this branch until after we cut the next release22:50
hatchjust to be safe :)22:50
hatchkadams54: 662 is ready for its final review and qa - but it won't be landed until after we cut the next release23:02
hatchso probably wednesdayish23:02
kadams54ok23:03
=== kadams54 is now known as kadams54-away

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