hazmat | dpb1, icon.svg is fetched from manage.jujucharms.com/store i believe | 01:09 |
---|---|---|
rick_h | dpb1: only reviewed charms will have the icon pulled. It'll provide a fallback icon depending on any categories set or use the default otherwise | 01:24 |
bcsaller | rick_h: ... and we're working again... | 01:27 |
rick_h | bcsaller: :) | 01:27 |
rick_h | bcsaller: but that was one good break I must say | 01:28 |
bcsaller | ha, yeah | 01:28 |
antdillon | gary_poster, Morning, I have some inspector updates if anyone was free to take a look? | 13:18 |
antdillon | gary_poster, lp:~ya-bo-ng/juju-gui/inspector-styling | 13:18 |
gary_poster | antdillon, great! yeah, I'll get them moving. Thanks! | 13:18 |
antdillon | gary_poster, That branch include pre and post deployment styling | 13:19 |
antdillon | includes* | 13:19 |
gary_poster | antdillon, awesome! the pre- has really needed some love. :-) | 13:19 |
hatch | antdillon, hey, saw your latest, looks good - I was wondering if you made any markup changes on the ghost to match the v11 mockup? | 13:26 |
gary_poster | jcsackett, https://codereview.appspot.com/11532044/ | 13:41 |
antdillon | hatch, Sorry the ghost? The only mark up chances I did was to the units list it add the icons | 13:42 |
hatch | the ghost-inspector | 13:42 |
hatch | but you answered the question anyways :) | 13:42 |
antdillon | hatch, Any major conflicts? | 13:43 |
hatch | well I'm going to have to change the markup today to match the latest mockup | 13:44 |
hatch | not an issue | 13:44 |
antdillon | hatch, Ok thanks, anything I can help with? | 13:45 |
antdillon | hatch, Added a few images too | 13:45 |
hatch | only if you want to change the markup now? :) | 13:46 |
hatch | it's ok I'm going to start on the header section pretty soon | 13:48 |
hatch | :) | 13:48 |
antdillon | hatch, Only a couple of lines in services.js and removed a class from serviceOverview.handlebars | 13:51 |
hatch | gotcha | 13:52 |
antdillon | hatch, Did update the buttons as I thought you guys might want the controlled as components not windows specific | 13:52 |
gary_poster | antdillon, landed | 14:00 |
hatch | jujugui - looking for two reviews https://codereview.appspot.com/11574043/ | 14:00 |
hatch | ok only one | 14:00 |
hatch | thanks Makyo | 14:00 |
antdillon | gary_poster, Is it on uistage? Just wanna check its all ok | 14:03 |
gary_poster | antdillon, will be up in 10 minutes | 14:05 |
bac | hatch: done | 14:06 |
antdillon | gary_poster, Great thanks | 14:11 |
rick_h | jujugui inspector refactor #1 reviews requested please https://codereview.appspot.com/11579043 | 14:30 |
gary_poster | antdillon, it is up | 14:31 |
gary_poster | should have been up 15 minutes ago; trigger failed for some reason | 14:31 |
dpb1 | rick_h: ok (on fetching charm icons from the store). shame. I guess I'll look into setting a category, since the default icon looks like a broken image. :) | 14:36 |
rick_h | dpb1: :/ that must be something different | 14:36 |
gary_poster | dpb1, what charm? or is this a local charm? | 14:38 |
rick_h | gary_poster: http://paste.mitechie.com/show/985/ is the diff | 14:38 |
dpb1 | gary_poster: our landscape charm, local for now. | 14:40 |
rick_h | dpb1: ah, in that case we've not landed the change that will load the default icon for local charms. It's on the todo list. | 14:40 |
gary_poster | what he said :-) | 14:41 |
dpb1 | rick_h, gary_poster: ya, here is what I'm staring at now. :) http://i.imagebanana.com/img/8klice7r/Selection_007.png | 14:43 |
gary_poster | dpb1, ack, ugly. We'll should get that addressed for next week. | 14:44 |
dpb1 | cool | 14:44 |
dpb1 | thx | 14:44 |
antdillon | gary_poster, Is the #service-menu still in the trunk? I've been asked to restyle. Is there a flag for that part? | 14:57 |
gary_poster | antdillon, it is, but currently it is not part of the serviceInspector flag. remove that from url and then click on a service and you will see it. We'll have to figure out how to incorporate it later, but if we have an idea of goal that will be a win | 14:58 |
antdillon | gary_poster, Sure just checking there wasn't a flag for it | 14:59 |
gary_poster | antdillon, luca, dunno if I missed anything from you. machine crashed | 15:10 |
luca | gary_poster: not from me, but | 15:11 |
luca | gary_poster: have you seen this: https://docs.google.com/a/canonical.com/file/d/0B7XG_QBXNwY1Tm9PVWhMZzB4alk/edit?usp=sharing | 15:11 |
antdillon | gary_poster, Nothing from me either | 15:13 |
gary_poster | cool | 15:14 |
gary_poster | luca, no. the point is the bottom right, I assume | 15:14 |
gary_poster | ? | 15:14 |
luca | gary_poster: yup | 15:14 |
luca | gary_poster: those are the canvas controls | 15:14 |
gary_poster | cool, thanks luca. is the "Import" a hover-over? | 15:14 |
gary_poster | the text I mean | 15:15 |
luca | gary_poster: yeah, they are "instant" tool tips, there should be no delay, they appear as soon as you hover over the icon | 15:15 |
luca | gary_poster: it shows the name and the keyboard shortcut | 15:15 |
gary_poster | cool, luca thanks very much for highlighting that to me. will be helpful, maybe/hopefully before you return :-) | 15:16 |
luca | gary_poster: hehe | 15:16 |
=== rogpeppe1 is now known as rogpeppe | ||
gary_poster | luca, landscape stuff goes in center or on left? | 15:19 |
luca | gary_poster: on the left preferably, only until alerts comes into play | 15:20 |
gary_poster | ok cool luca, thanks | 15:20 |
Makyo | jujugui 2 reviews for caching charms in the inspector. https://codereview.appspot.com/11526044 | 15:48 |
rick_h | bcsaller: hatch lp:~rharding/juju-gui/reorg-inspector-module | 15:59 |
gary_poster | hey luca or antdillon could you give us the assets for the details/constraints/config icons? I don't see them. Even if they aren't right they will be nicer than letters :-) | 16:06 |
luca | gary_poster: ant is in a meeting but I'll get him to send them over asap | 16:18 |
gary_poster | luca, cool. main goal is before you leave on vacation and he leaves gui for a while--so, before today EoD ;-) | 16:18 |
antdillon | gary_poster, Luca will get them over to you | 16:44 |
luca | gary_poster: do you want them as sag or png? | 16:45 |
luca | gary_poster: svy^ | 16:45 |
luca | bah | 16:46 |
luca | gary_poster: you know what I mean :P | 16:46 |
antdillon | gary_poster, I have a new branch with new styles for the environment menu but have also removed the view link which opens the inspector. The inspector is due to open on click | 16:46 |
antdillon | gary_poster, Is that developed on the latest? I have had a look through the code for a function to open the correct inspector but didnt see it | 16:48 |
antdillon | gary_poster, Is it ok the land the restyle with a none fault? | 16:49 |
=== anthonydillon is now known as antdillon | ||
gary_poster | antdillon, sorry lunch | 17:13 |
gary_poster | back now | 17:14 |
antdillon | gary_poster, No prob | 17:14 |
gary_poster | luca, png | 17:14 |
gary_poster | luca, but I'll take what I can get | 17:14 |
gary_poster | :-) | 17:14 |
antdillon | gary_poster, Its sent as png | 17:15 |
antdillon | gary_poster, My branch has a known fault but has the styles are right | 17:16 |
gary_poster | antdillon, perfect thanks. as to your question, we will have to adapt the code somehow. I'll take your branch and we'll figure out what to do; the most important thing is to see how it works | 17:16 |
gary_poster | antdillon, sounds perfect. the known fault is that it shows up without the inspector, right? | 17:16 |
antdillon | gary_poster, You can take my branch and all you'll have to add is the inspector is opened when clicked on a deployed service block | 17:17 |
gary_poster | antdillon, perfect. same lp branch as before? | 17:18 |
antdillon | gary_poster, No, i you deploy a service block then click on it you get a menu. In the menu is "View" this has been removed as the inspector should to opened on click | 17:18 |
antdillon | gary_poster, No let me get it for you | 17:18 |
gary_poster | k | 17:18 |
antdillon | gary_poster, lp:~ya-bo-ng/juju-gui/styling-enviroment-menu | 17:18 |
gary_poster | antdillon, thanks so much! thanks also generally for your gui help. we'll miss you. :-) have a great weekend | 17:19 |
antdillon | gary_poster, This behaviour already exist on the serviceInspector flag so should be fine right? | 17:20 |
gary_poster | antdillon, yeah we will have to tweak some things, no worries | 17:20 |
antdillon | gary_poster, lol I enjoyed it, beats boring html and css all day | 17:20 |
gary_poster | :-) cool antdillon | 17:20 |
antdillon | gary_poster and team Thanks for all the help | 17:21 |
gary_poster | they are all still at lunch ;-) but they appreciate also | 17:21 |
gary_poster | ttyl | 17:21 |
=== matsubara is now known as matsubara-lunch | ||
gary_poster | jujugui, please review https://codereview.appspot.com/11593043 | 17:53 |
bac | o/ | 17:54 |
benji | gary_poster: I'll take one | 17:54 |
jcastro | heya gary_poster | 18:01 |
jcastro | in the new UI | 18:01 |
jcastro | I am confused a bit | 18:01 |
jcastro | on how to make the unit go into "relation mode" by showing the line | 18:01 |
jcastro | and dragging the box around | 18:01 |
jcastro | do I need to click and hold on a specific place? | 18:01 |
gary_poster | jcastro, is this with or without the service inspector (:flags:/serviceInspector in url)? | 18:02 |
jcastro | without | 18:02 |
jcastro | It "feels" like making it show the line to connect to another box is trickier than before | 18:02 |
jcastro | I have no other data other than "it's way harder than before" | 18:02 |
jcastro | especially on a touchpad. | 18:02 |
gary_poster | jcastro, the way that is supposed to be discoverable is that you click on a service and the menu shows up | 18:03 |
jcastro | oh! | 18:03 |
jcastro | dude, waaaaaay better | 18:03 |
jcastro | ok | 18:03 |
jcastro | so I shouldn't even try to click and hold, just click, gotcha | 18:03 |
gary_poster | yeah | 18:03 |
gary_poster | cool, glad that helps :-) | 18:04 |
jcastro | man this is scary | 18:05 |
jcastro | I set up like 3 dual-unit stacks in the UI in like 30 seconds | 18:05 |
rick_h | hatch: http://www.flickr.com/photos/7508761@N03/9103189029/ | 18:14 |
rick_h | hatch: http://flickr.com/gp/7508761@N03/G18R77 that should work | 18:16 |
Makyo | rick_h, https://codereview.appspot.com/11526044/ | 18:26 |
bcsaller | rick_h: http://pastebin.com/0cH7Zrbc still has a console.log in it, but you'll get the point | 18:40 |
=== matsubara-lunch is now known as matsubara | ||
bcsaller | https://codereview.appspot.com/11433048/ should properly fix the undinbind issues | 19:00 |
rick_h | jujugui quick side branch to make 'one true store' https://codereview.appspot.com/11598043 | 19:02 |
Makyo | jcsackett, https://twitter.com/BriDaHypnoKitty/status/358297310465257473/photo/1 | 19:25 |
Makyo | hatch, http://i.imgur.com/sg017lt.gif | 19:49 |
bcsaller | http://www.beerknurd.com/stores/raleigh/ has a nice beer list for tonight, might go there with my cousin if people want to come along | 20:31 |
hatch | jujugui looking for two reviews https://codereview.appspot.com/11607043 | 20:33 |
jcsackett | jujugui: two reviews, aggressive QA: https://codereview.appspot.com/11611043 | 21:30 |
jcsackett | please. :-) | 21:30 |
rick_h | http://www.landofdroid.com/2013/exclusive-images-of-the-ubuntu-edge/ | 21:45 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!