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