/srv/irclogs.ubuntu.com/2009/09/18/#launchpad-reviews.txt

EdwinGrubbswgrant: thanks. I guess it can go away if bug 315358 is ever completed.00:00
mupBug #315358: Expose the storeblob service in API <api> <Launchpad Bugs:Triaged> <https://launchpad.net/bugs/315358>00:00
wgrantEdwinGrubbs: Mmmmm, possibly.00:01
wgrantEdwinGrubbs: But the API is awkward right now.00:01
wgrantEdwinGrubbs: apport has broken once or twice in the past when field names changed on that page. I think there's a test for that now, but best to just check that manually..00:03
EdwinGrubbswgrant: we're just changing the template which defines the header. So it would be hard to imagine how it could break.00:04
wgrantEdwinGrubbs: That sounds safe, yes.00:05
=== jml changed the topic of #launchpad-reviews to: on call: jml || reviewing: - || queue: []
thumperjml: you on call?04:05
jmlthumper, yes.04:06
thumperjml: bug 40764304:06
mupBug #407643: CodeImportNewView confusing after form changes <Launchpad Bazaar Integration:In Progress by thumper> <https://launchpad.net/bugs/407643>04:06
thumperjml: take a look at the attached image04:06
thumperjml: the fix is all in the page template04:06
thumperbranch is pushed04:06
thumperI'm awaiting the scan04:06
thumperI'd be tempted to rs it myself04:07
thumperit is very simple04:07
thumperno test changes needed04:07
thumperit just indents the fields04:07
jmlthumper, the screenshot looks fine to me.04:08
jmlthumper, I'd still like to have a poke at the code04:08
thumperjml: it is closer to what it used to be04:08
thumperjml: I've proposed for merging with you as reviewer04:08
jmlthumper, thansk.04:10
jmlmwhudson, I've replied to your review06:07
=== jml changed the topic of #launchpad-reviews to: on call: jml || reviewing: [sinzui] || queue: []
mwhudsonjml: woo06:12
jmlsinzui, you too.06:24
=== jml changed the topic of #launchpad-reviews to: on call: jml || reviewing: [] || queue: []
mwhudsonjml: replied06:36
jmlmwhudson, thanks.06:38
=== noodles775 changed the topic of #launchpad-reviews to: on call: jml || reviewing: [] || queue: [noodles]
=== noodles775 changed the topic of #launchpad-reviews to: on call: - || reviewing: [] || queue: [noodles]
noodles775I'm guessing jml has finished reviewing for the day :)09:29
jmlI has.09:30
allenapnoodles775: Fancy a incredibly trivial one-line review?09:36
allenaphttps://code.edge.launchpad.net/~allenap/launchpad/rocketfuel-get-overwrite/+merge/1203809:36
noodles775allenap: sure - I'd ask for a swap but mine's 500 lines :/09:36
allenapnoodles775: Heh :)09:36
allenapnoodles775: The diff is going to take a while, so here's one I prepared earlier: http://pastebin.ubuntu.com/273315/09:37
noodles775Cool.09:37
* noodles775 reads the emails.09:38
noodles775allenap: so your branch will result in *all* branches updated including the overwrite flag right? Are there any side-effects or dangers in doing this?09:42
allenapnoodles775: If someone has edited one of their sourcecode branches it will be clobbered.09:42
allenapnoodles775: But people - including me - shouldn't really be making a habit of that, because if I forget to revert I might get strange behaviour in other branches.09:43
noodles775allenap: ah, so update_branches() is only ever called in update_sourcecode() - right.09:44
noodles775Yep.09:44
allenapnoodles775: Yep, I think so.09:44
noodles775Yep, it is (currently).09:44
noodles775allenap: r=me sent.09:45
allenapnoodles775: Thanks!09:45
noodles775np.09:45
wgrantjml: Did you get around to the update-sourcecode fixes, or should I do it now I've time?10:03
=== henninge_ is now known as henninge
=== henninge is now known as henninge-lunch
=== henninge-lunch is now known as henninge
noodles775adeuring: are you available for reviews today?12:55
=== noodles775 changed the topic of #launchpad-reviews to: on call: - || reviewing: [] || queue: [noodles, noodles]
adeuringnoodles775: i'm afraid that i would produce quite some nonsense today: have th fl12:56
adeuringthre flu12:56
noodles775adeuring: np! Hope you get better soon!12:56
adeuringnoodles775: thanks!12:56
=== danilo_ is now known as danilos
=== mrevell is now known as mrevell-lunch
=== mrevell-lunch is now known as mrevell
jmlwgrant, I didn't get around to it.13:53
wgrantjml: OK, thanks. I'll do it tomorrow.13:53
=== noodles775 changed the topic of #launchpad-reviews to: on call: - || reviewing: [] || queue: [noodles, noodles, noodles]
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [] || queue: [noodles, noodles, noodles]
barrylooks like it might be noodles for breakfast?!14:05
noodles775heh, barry, could you pls look at this one first:14:06
noodles775https://code.launchpad.net/~michael.nelson/launchpad/432416-binary-pkg-index-oops/+merge/1205114:06
noodles775it's the most urgent of my three (and related to one you ui-reviewed for me yesterday).14:06
barrynoodles775: sure thing14:07
noodles775Thanks!14:07
noodles775barry: I'm just going to take a break for 20 mins.14:07
barrynoodles775: no worries14:07
BjornTbarry: i have a branch that needs reviewing as well. it's non-urgent, but also really simple (all it does is to add a test)14:13
=== BjornT changed the topic of #launchpad-reviews to: on call: barry || reviewing: [] || queue: [noodles, noodles, noodles, BjornT]
barryBjornT: cool.  salgado should be joining me here today, but i think abel is still out14:13
salgadobarry, I don't think I'll be able to join you, sorry.  I've got 171 tests failing on my breadcrumbs branch14:16
barrysalgado: double :(14:17
barrysalgado: good luck!14:18
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [noodles, noodles, BjornT]
noodles775I'm back barry if you have any questions etc.14:32
barryk14:32
gmbnoodles775: Do you have the time to give me a UI review?14:33
noodles775gmb: yep, add me as a ui-reviewer - I'll try to look at it in the next hour.14:34
gmbnoodles775: Awesome, thanks.14:35
noodles775henninge: it's looking great! I've just merged so I can take a quick look at the issues you identified.14:40
henningenoodles775: thanks. I am currently adding icons to the "What's Launchpad" list.14:40
abentleybarry: I can has review? https://code.edge.launchpad.net/~abentley/launchpad/restyle-subscriptions/+merge/1200814:42
noodles775henninge: so you can left-align the stats simply by removing the text-align: center from #homepage-stats in style.css.14:42
henningenoodles775: it is meant to left-align with the left edge of the search field.14:43
henningeat least that's what the mock-up looks like but I am not sure Martin did that on purpose.14:44
noodles775henninge: and it does doesn't it? (I'm juts disabling the rule and it seems to do exactly that?)14:44
noodles775or did I misunderstand?14:44
henningeI am trying now .. ;)14:44
henningewow, it (almost) works!14:45
henningestill have to adjust the width.14:45
henningenoodles775: I tried a few things this morning but this must be a combination I missed ... ;-)14:46
noodles775henninge: aha. Also the 'Take the tour' is pointing to +newteam for some reason.14:46
henningeoops, copy-and-paste ...14:47
barryabentley: put it on the list!14:47
henningenoodles775: wait, which one?14:47
abentleybarry: Okay.  I always ask first.  Should I not do that?14:47
henningethere are two, one left, one right.14:47
=== abentley changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [noodles, noodles, BjornT, abentley]
henningeah, got it, noodles775 14:48
barryabentley: as the day wears on, it's gets better to ask :)14:48
barry(or as the queue grows)14:49
EdwinGrubbsbarry: can you review https://code.edge.launchpad.net/~edwin-grubbs/launchpad/featured-projects-ui-3/+merge/1205314:51
barryEdwinGrubbs: you can put it on the queue, but it's going to be a while.14:51
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [noodles, noodles, BjornT, abentley, Edwin]
noodles775henninge: and the un-clickable login/logout/user is because the #locationbar seems to have a calculated height of 0px... still trying to find out why. Explicitly setting it to 1em gets around it, but better to find the real cause.14:55
henningenoodles775: oh, wgrant pointed me to a bug about it!14:56
henningenoodles775: Bug #42924714:56
mupBug #429247: Locationless <h1>s block login/out widgets <story-ui-3> <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/429247>14:56
noodles775henninge: great, I just updated it with that info.14:57
noodles775Thanks!14:57
jtvWho broke our front page?14:58
jtvnoodles775?14:58
noodles775jtv: probably - if it's looking for isBetaUser then yes :/... 14:59
* noodles775 looks14:59
noodles775jtv: huh? what's the problem that you're seeing?14:59
jtvnoodles775: something it uses obviously tries isBetaUser, yes14:59
noodles775jtv: barry is reviewing a fix for that now... but I don't see it on the front page... ah, the translations front page.15:00
jtvI guess that page got the application tabs through means other than the way your check expected15:00
jtvYes.15:00
noodles775jtv: More likely your view does not inherit from LPView... checking now.15:00
jtvIt's more than just our front page, actually.  :/15:01
noodles775jtv: it's any view that does not inherit from LPView...15:01
noodles775jtv: I wrongly assumed that all non-locationless pages in LP inherit from LPView - but there are *heaps* of old views that don't. I should have checked... sorry.15:02
jtvI figured.  And the pages apparently get their application tabs through means other than what the check expected15:02
noodles775jtv: the fix is simple, hopefully I can land it soon.15:02
jtvMaybe worth updating those views as well, sometime after release.15:03
jtvShould I file bugs?15:03
noodles775jtv: no, the fix is a general one...15:04
jtvI meant about making views inherit from LaunchpadView.  But that too may be best done automatically and centrally.15:04
noodles775jtv: oh, perhaps. But yes, as a general bug, I've got a list here: http://pastebin.ubuntu.com/273421/15:05
jtvnoodles775: cool...  if/when we really want to do something about it, I guess we could correlate that list of class names to stuff we find in zcml.15:06
rockstarbarry, if your queue still looks like this after I get these two branches landed, I'd be happy to help out.15:09
barryrockstar: awesome, thanks15:11
barrynoodles775: first review sent15:12
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [noodles, BjornT, abentley, Edwin]
noodles775Thanks barry.15:12
barrynoodles775: see what you think of my proposed alternative15:12
noodles775barry: I'll give it a try... one tic.15:13
henningenoodles775: I quite like setting locationbar to 1.5em...15:14
noodles775henninge: great, it fixes the issue, so might be a good solution in and of itself.15:15
henningenoodles775: it looks good on all pages, I always thought they had too little space at the top.15:15
henninge1em is enough, though.15:18
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [noodles, BjornT, abentley, Edwin, Edwin]
intellectronicabarry: can i bother you for a quick ui review? martin has already given a thumbs up for a previous version, so this is just a last check before i can land15:25
intellectronicabarry: https://devpad.canonical.com/~tom/bugs-homepage-redesign-final.png15:26
noodles775gmb: sorry, I'm not going to be able to get to that ui review before I run... you might be best adding barry and popping it on the queue (retrospectively) ;D15:26
gmbnoodles775: Righto, thanks anyway.15:26
gmbbarry: Is that okay with you ^^?15:26
henningenoodles775: I pushed a new version.15:27
intellectronicagmb: i can ui review15:27
gmbintellectronica: Good point. I was going to swap with your for code review anyway.15:27
intellectronicagmb: yes indeed :)15:27
henningenoodles775: if you are fine with it, I'll leave it as it is now (except for upodating the blog posts).15:27
gmbintellectronica: https://code.edge.launchpad.net/~gmb/launchpad/bugtask-index-conversion/+merge/12054, if you would. I still need to fix some tests before you review the code changes; I'll let you know when I'm done.15:28
barryi'm happy to do quick ui reviews between code reviews if there are screenshots15:28
intellectronicagmb: ditto (still doing tests before i can submit for code reviews)15:28
gmbintellectronica: RIght.15:28
gmbintellectronica: Oh, one point: I know I still need an icon for the Convert to question action; that's my next job.15:29
intellectronicagmb: and ditto, if you have a screenshot i can ui review now quickly15:29
gmbintellectronica: https://devpad.canonical.com/~gbinns/screenshot.png15:29
intellectronicagmb: a smiley will do ;)15:29
barrygmb: i can do a ui review.  paste the screenshot url15:29
gmbNow now, no fighting please...15:29
barryintellectronica: what am i looking for?  the design of the whole page or specific parts of it?15:29
intellectronicabarry: both. the general concept martin already reviewed, but i just want a last check that everything looks sane15:30
intellectronicagmb: looks very nice. it will be _very_ cool if we could get the description/comments/postbox aligned. is it unrealistic given the time, though?15:31
barryintellectronica: i love certain this about it.  the fact taht you can change the supervisor and contact right there (are they inline edits?) and the wonderful human friendly breadcrumbs15:31
intellectronicagmb: the remove cve link should be on the next line15:32
gmbintellectronica: I've played with it and so far have had no joy. Unless you can think of something off the top of your head that might work...15:32
gmbintellectronica: Argh. Missed that. Thanks.15:32
barryintellectronica: the only thing that looks weird to me is the tag cloud.  i vaguely feel like it should be left-aligned or justified, but if that's difficult it needn't hold up the branch15:32
intellectronicabarry: they're not inline, and you could change them from that page in 2.0 (but i guess if you noticed now it means that my plan to make the links more discoverable worked)15:32
barryintellectronica: definitely15:33
intellectronicabarry: i like the idea of justifying! do you happen to know the css for doing that?15:33
intellectronicagmb: b.t.w if you can't come up with an icon for converting to question i think you can use (+)15:34
gmbintellectronica: Right, that was my fallback plan.15:34
barryintellectronica: i think: style="text-align: justify;"15:34
gmbintellectronica: I'm filing a lot of small tweaks under "fix it later"15:35
noodles775barry: your idea works well and is much cleaner... doing an incremental now.15:35
barrynoodles775: rawk15:35
deryckintellectronica, gmb -- for description to stretch out and line up to the side bar on a bug page requires a bug fix from me to the widget, which I can't get today.  But comments could line up to side bar like the table, I would think.15:35
barryintellectronica: if that looks good, ui=me* (but i'm also happy to look at it again if you're not sure)15:36
intellectronicagmb: the alignment of the main area, especially now that the cves are there, is pretty ugly. but i guess it's quite a lot of work to fix that now15:36
barrygmb: did you have a ui you wanted me to look at?15:36
gmbderyck: But that then goes against all the other pages' layouts (where commetns are quite compact)15:36
gmbbarry: An extra set of eyes is always helpful; feel free.15:36
deryckgmb, ah, ok.  I didn't realize we had a rule/pattern for that.  If comments are fixed, I could make the widget adjust to the comments pretty easily.15:37
deryckfixed is easy, fluid, not so much15:37
barrygmb: did you have to fill the heading slot to get the editable h1?15:37
intellectronicabarry: that was such a good idea. refresh and see15:38
gmbderyck: Right. So, under main_side, the boardComments isn't fixed width... I think it just has a big right margin.15:38
barryintellectronica: beautiful, thanks!15:38
gmbbarry: Yes.15:38
intellectronicabarry: thank you15:38
deryckgmb, hmmm, ok, so that may be hard then, too, if that comments are fluid.15:39
intellectronicabarry: b.t.w i'm going to interpret your ui* and martin's review from yesterday as ui-approved15:39
gmbintellectronica, deryck, barry: I just have to step out for a minute (*must* post some documents today and I've forgotten)... I'll address any comments when I return.15:40
noodles775jtv: mthaddon updated the configs temporarily on edge so that your pages won't oops.15:40
deryckgmb, cool15:40
jtvnoodles775: acknowledged with thanks15:40
mrevellhenninge: ping15:42
henningemrevell: here15:42
mrevellHi henninge, I'm just looking at your mock-up.15:42
henningemrevell: its not a mock-up15:42
henningeit's a screenshot of my page.15:42
henninges/page/branch/15:43
mrevellsorry, yes15:43
henningemrevell: just saying, the design work (Martin) and the coding (Me) is mostly done, I am just wondering about content and wording now.15:43
henningemrevell: and I need the blogposts that will ve released next week ;-)15:44
henningebe15:44
mrevellhenninge: It may be too late to change but I think this design gives far too much prominence to the "latest blog posts" and not enough to getting started. If I take a look at, say, GitHub's home page I fe15:45
mrevellhenninge: You need next week's blog posts? Surely it's taking a live feed, isn't it?15:45
henningemrevell: no, it's not ...15:46
henningemrevell: these are just copied from the current page which were hard-coded, too.15:46
barrygmb: could you do me a favor?15:47
barrygmb: file a bug on the fact that you had to override the heading slot to do that?  i'd like to fix that post-3.015:47
henningemrevell: If I had a function I could call to get the blogposts, I could easily put them on the page15:47
mrevellhenninge: That's a damn shame: we're just giving "what's new" a fancy new, and inaccurate, name. I think it really must not be "Latest blog posts" as that will very quickly become inaccurate. We'll continue to make blog posts between releases.15:47
henningebut coding the polling now seems work15:47
barrygmb: other comments: the "link to CVE/Remove CVE links.  why are both visible?  and if they need to be, why aren't they on separate lines?15:48
henningemrevell: yes, I was wondering about that. So we stick with "What's new" for now until we get the polling coded.15:48
mrevellhenninge: If that's how it has to be, I guess so.15:48
barrygmb: shouldn't "Convert to question" have an icon of some sort?  it stands out as odd without it15:48
mrevellhenninge: And I'll get that list of blog posts to you, in that case.15:49
henningemrevell: please15:49
barrygmb: i wonder if the elements in the left-ish portlet under the bug summaries shouldn't be as wide as the bug summaries?15:49
barrygmb: EOT15:49
barrynoodles775: which branch of yours am i reviewing next?15:50
mrevellhenninge: So, wrt to the design, I'm not sure what to suggest. I liked the initial mock-up but now I see actually coded up I think it lacks the punch we need for our home page. Perhaps the problem is that each section seems to have equal emphasis because the headlines and body text in each segment has the same font size. On the "not logged in" version I'd suggest that the "Launchpad is a..." section should probably have a subtle 15:52
mrevellbackground and larger headline15:52
=== sinzui changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [noodles, BjornT, abentley, Edwin, Edwin, sinzui]
henningemrevell: ok, kfogel suggested something like that, too.15:52
* barry will just pick one15:53
mrevellhenninge: For the logged in version, I feel like we should give more prominence to "Getting started" than to "What's new". perhaps swap them?15:54
henningemrevell: It would be easiest to swap the columns.15:55
henningenot just one portlet.15:55
noodles775barry: either is fine, if you want an easy one, the sprint templates (they're probably more urgent I'd guess).15:55
barrynoodles775: i'll take sprint-index-and-attend15:56
mrevellhenninge: I think that would work well.15:56
mrevellhenninge: henninge: And this is perhaps for another time but I like the fact that GitHub has a "Recently updated repositories" section on its home page. I wonder if we could do something similar.15:56
noodles775barry: I have to run now, so feel free to prioritise the last one as you see fit :)15:56
barrynoodles775: gotcha15:56
noodles775barry: btw, did you see the diff I just sent for the first one?15:56
henningemrevell: I am sure the code guys could give us that data.15:57
* noodles775 is about to run ec2test, and will do so to land if you're happy with it.15:57
henningemrevell: I mean, live, through some function call.15:57
mrevellhenninge: Well, I was thinking not so much just code but maybe something more universal. I'll add that as a comment on the bug, as it's probably not possible to do right now :)15:57
barrynoodles775: looks fine.  i'll update the mp.  my comment about those closing braces is that we're /not/ consistent so either way is probably fine for your branch ;)15:58
noodles775barry: great, thanks!15:58
henningemrevell: oh, latest bug, branch, question, etc ... ?15:59
henningemrevell: yes, but not now. ;)15:59
mrevellhenninge: Yeah, too much work for right now :)15:59
mrevellhenninge: So, as for text...16:01
mrevellhenninge: minor quibble: I think "Read the user guide" works better than "Read the manual", but I'm easy if it doesn't fit.16:02
henningeshould16:02
mrevellhenninge: Going back to the design, I wonder if we should better emphasise the search box. Take a look at github.com and sourceforge.net and you'll see that the eye's immediately drawn there, whereas I find it easy to skip over ours.16:04
henningemrevell: yes, both have them graphically enhanced and placed top-left.16:05
henningemrevell: maybe the centering is a bad idea.16:05
barrynoodles775: ping16:06
mrevellhenninge: I don't mind the centering, but I think the graphical enhancement works. At the moment, our home page looks a bit like the poor cousin of our two main rivals.16:06
noodles775barry: yep?16:06
barrynoodles775: this page doesn't look quite right: http://people.canonical.com/~michaeln/tmp/sprint-index-after.png16:06
barrynoodles775: you've got an h1 above the app tabs and no heading above the breadcrumbs16:06
mrevellhenninge: Our home page is quite text heavy, which is fine, but I think we need to throw in some more clues to show people where to look.16:07
noodles775barry: yes, I meant to mention that in the MP, sorry...16:07
mrevellhenninge: I'm not sure, though, that we'll get all that fixed in time for 3.016:07
noodles775barry: afaics, the breadcrumbs should not be appearing according to the heading rules?16:07
=== bigjools changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [noodles, BjornT, abentley, Edwin, Edwin, sinzui, bigjools]
* bigjools sees the queue and cries16:07
henningemrevell: we won't.16:08
barrysigh16:08
noodles775barry: not sure what you got, but:16:08
noodles775<noodles775> barry: yes, I meant to mention that in the MP, sorry...16:08
henningemrevell: even adding a background takes some work to look good and not cheap.16:08
noodles775<noodles775> barry: afaics, the breadcrumbs should not be appearing according to the heading rules?16:08
mrevellhenninge: Right16:08
henningei just tried and ...16:08
barrynoodles775: this should be a root context index view, so i think that's right.  there should be no breadcrumbs16:09
mrevellhenninge: I think we should probably spend some more time looking at the design of the page and maybe involve our friends in the User Experience and Design team at Canonical.16:09
henningereminded me of earlier years of the web 16:09
mrevellhenninge: but that's for another day, obviously16:09
henningemrevell: so the question is: go with this now or leave the old home page in place (minus old navigation).16:09
noodles775barry: yep. Any other questions before I run out the door? :)16:10
barrynoodles775: that's it for the ui.  everything else looks great.  i'll mention this in my comments16:10
noodles775barry: btw, I'm assuming that's a bug in the heading stuff, or is there something I can do about it?16:10
noodles775barry: ok, great, thanks!16:10
mrevellhenninge: Hmm, big question! I like the direction of beuno's mock-up but actually seeing it in screenshots I prefer what we have now. I am not, though, in any position to make that decision alone haha16:11
barrynoodles775: that's an interesting question!  this hasn't come up before.  you might ping salgado about that (it's a +hierarchy adapter issue).  if he doesn't have an ideas off the top of his let, i'll try to think about it16:11
henningemrevell: well, who is?16:11
mrevellhenninge: At a guess, flacoste16:12
henningemrevell: he just saw it and said it was fine ... ;-)16:12
mrevellhenninge: In which case, I'll shut up16:12
mrevell:)16:12
henninge;)16:12
mrevellhenninge: I think, though, that after the release we should revisit the home page. I'll post to the list with some idea of what I'm thinking about.16:12
henningemrevell: but please remember that blog post list. When do you think I can have it?16:13
mrevellhenninge: I'll send something over now.16:13
henningecool16:14
gmbbarry: So, in order: 1) Yes, I'll file that now; 2) They both have to be visible because you can both add and remove CVEs (if there's no CVE linked the 'Remove' link disappears. The separate lines thing is now fixed; 3) Yes, Convert to question needs an icon - that's my next job after fixing test failures; 4) Er... I don't think I understand what you mean by "left-ish portlet"16:14
=== EdwinGrubbs is now known as Edwin-afk
barrygmb: 1-3) cool; 4) "left-ish portal" means the bug description and everything below that.  see how there's a lot of vertical whitespace below the summaries and in between the description/comments and the right portal?  that's what i think should be collapsed16:17
gmbbarry: I agree; that looks ugly. I haven't (yet) found a way to fix it though. I'll keep trying, but in the interests of getting at least *something* landed before PQM closes, would you be okay with me filing a bug and fixing it later (assuming there isn't a cheap fix to be had now).16:18
intellectronicagmb: guess what, i also just realised that i need to override the page title. how?16:19
barrygmb: yes.  ask sinzui though.  he did something related (added some css to stretch things the full page width).  if he can't help come up with something simple, file the bug and land it16:19
barrygmb: er, well, by land it i mean ui=me* :)16:20
sinzuibarry: gmb and I have talked. My tricks did not help with his real problems16:20
barrysinzui: k.  gmb file that bug16:20
barrygmb: is this bugtask-index-conversion?16:21
intellectronicagmb: did you find out how to override the page title?16:23
intellectronicabarry: maybe you know? ^^^16:23
gmbintellectronica: Yes16:23
gmbintellectronica: Add a page_title property to your view16:23
henningemrevell: the column swapping is not as easy as I thouhgt it was so I am sorry but we'll have to postpone that, too...16:23
gmbRemove the pagetitles entry16:24
mrevellhenninge: okay16:24
intellectronicagmb: lovely, thanks16:24
sinzuigmb: barry: I think the problem requires a very invasive redesign of the base-layout. If we can remove the heading-slot. Then we can make a simple slot that is for content before the sidebar.16:24
gmbintellectronica: And then add     override_title_breadcrumbs = True to your view class.16:24
intellectronicaoh ok16:24
gmbintellectronica: That last bit stumped me :)16:24
barrysinzui: i would love to remove the heading slot.  but yeah, post-30 :)16:25
barryintellectronica: i hope you don't need to override the whole <title> though16:25
gmbbarry: Thanks.16:25
barryintellectronica: do you just need to make it human readable?16:25
intellectronicabarry: i'm not sure what you mean by human readable. it changed from what it used to be to 'something : something'16:26
barryintellectronica: right.  human readable means, no +foo thingies16:27
intellectronicathe override_title_breadcrumbs = True trick worked, so i'm satisfied16:27
barryintellectronica: isn't the reverse breadcrumbs what you want?16:27
intellectronicabarry: no, because the title is a bit more verbose than the breadcrumbs16:28
barryintellectronica: hmm.  screenshot?16:28
abentleyrockstar: why aren't edit_whiteboard and edit_import @enabled_with_permission?16:29
intellectronicabarry: breadcrumbs is 'Bugs in firefox'. page title should be (and is now) 'Bugs in Mozilla Firefox'16:29
rockstarabentley, ah, they probably should be as well.16:29
rockstarabentley, edit_whiteboard doesn't need to be, since, if it's an import branch, anyone can write on the whiteboard (thus the problem with whiteboards)16:30
bacbarry: can i add a 35 line branch to your queue?16:30
abentleyrockstar: Do you really want to put lolspeak in lib/lp/code/browser/configure.zcml ?16:31
rockstarabentley, where?16:31
barryintellectronica: is that the full title or just the last component?  because salgado's branch is/will make it so you only need to specify page_title w/o the override.  but if the <title> is now what you want, then go for it.  we might need to re-address this later16:31
bacsinzui: you have two minutes to do the review for bug 432163?16:31
mupBug #432163: Team page Join link thinks it is a download button <trivial> <ui> <Launchpad Registry:In Progress by bac> <https://launchpad.net/bugs/432163>16:31
sinzuibac: yes16:31
barrybac: you can add it, but we'll see ;)16:31
bacsinzui: https://code.edge.launchpad.net/~bac/launchpad/bug-432163-join-link/+merge/1206416:32
abentleyrockstar: "<!-- NEW STUFFS GOES HERES -->" near line 36916:32
bacbarry: nm, sinzui is going to take it.16:32
rockstarabentley, ah, yea, I need to remove that now.16:32
rockstarabentley, and the LOLSPEAK was unintentional.  :)16:32
intellectronicabarry: yeah, the <title> is now what i want, so i won't worry about it until salgado lands his branch16:32
barrybac, intellectronica: coolio16:33
sinzuibac: r=me16:33
bacsinzui: thanks16:34
barryokay, i'm going back to code review now, so no ui reviews for a bit :)16:34
rockstarIs PQM closing today or Monday?16:34
abentleyrockstar: It it current behaviour to hide merges for import branches?16:36
rockstarabentley, no.16:36
abentleyrockstar: Were you aware that some projects are deliberately using merges on vcs-import branches to do code review of the upstream branches?16:37
henningemrevell: #homepage-whatslaunchpad ul 16:37
henningemrevell: I increased the font sizes on the headings and for the "What's Launchpad" section.16:38
rockstarabentley, oh wait.  The behavior is that you can propose merges into vcs-imports, but you can't propose vcs-imports for merging.16:38
rockstarEr, it's supposed to be.  I got a little overzealous.16:38
henningemrevell: http://people.canonical.com/~henninge/screenshots/launchpad-homepage-3.png16:38
abentleyrockstar: Okay, please fix.16:39
rockstarabentley, okay.  Please note that on the review.16:39
bacsinzui: i'm going to piggyback bug 432427 with the other.  it was a one line change16:42
mupBug #432427: Person's Code participation arrow is blueprint-coloured <trivial> <Launchpad Registry:Triaged by bac> <https://launchpad.net/bugs/432427>16:42
abentleyrockstar: What would you think about moving registrant below review team?  It seems like rarely-used information.16:42
rockstarabentley, well, technically, it should go away.16:43
sinzuibac: +116:43
abentleyrockstar: Onto a "wow, you're nitpicky" page?16:43
rockstarabentley, no, there was supposed to be a slot to put that info i.n16:44
abentleyrockstar: I don't understand.16:44
rockstarabentley, well, "registered by" is SUPPOSED to go up in the right floating next to the title.  However, I don't think the slot exists.16:45
abentleyrockstar: I see.16:46
abentleyrockstar: Should the import URL of an import branch be displayed similarly to the mirror url of a mirror branch?16:47
rockstarabentley, I thought about it, but I decided against it because import branches don't have just urls (like in the case of CVS).16:47
abentleyrockstar: Okay.  Maybe in the future we can display import data in the same place, whether it's a URL or a URL + branchname combo.16:49
rockstarabentley, yeah.16:50
mrevellhenninge: Looks good. I'm just trying to come up with summaries for these blog posts for you16:50
henningemrevell: yeah, make'm good. They don't have to be that short and concise anymore, remember? ;-)16:51
abentleyrockstar: in branch-management.pt, why do you have this nested tal:condition at the top?16:51
=== salgado is now known as salgado-lunch
abentleyrockstar: Isn't there a way to render a Link as HTML directly?16:54
rockstarabentley, what do you mean?16:54
abentleyI mean not <a id="codebrowse-link" class="sprite info" tal:attributes="href link/url" tal:content="link/text" >Browse this branch</a>16:55
abentleyrockstar: Maybe <a tal:replace=link/fmt:link> or something?16:56
rockstarabentley, ah, well, I would normally do that, but I need them rendered the way they are now when I attach javascript to them.16:59
abentleyrockstar: In order to customize the id?16:59
rockstarabentley, basically, we don't have an API to give the link an id.  *groan*16:59
abentleyrockstar: This is a DRY violation, so we should fix it.17:00
abentleyrockstar: Not necessarily in this branch, though.17:00
barrynoodles775: i think i fixed sprint-index.  i'll include the diff in my review17:00
=== matsubara is now known as matsubara-lunch
rockstarabentley, yeah, the problem is known.17:02
abentleyrockstar: in branch-management.pt, why do you have this nested tal:condition at the top?17:04
* rockstar looks17:04
rockstarabentley, I think that was a result of moving stuff around.  I'll remove the first condition.17:06
rockstarabentley, please note that in the review.17:06
abentleyrockstar: Cool17:06
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [] || queue: [noodles, BjornT, abentley, Edwin, Edwin, sinzui, bigjools]
* barry -> lunch, then more reviews17:12
abentleyrockstar: r=me, subject to the changes we agreed on.17:15
rockstarabentley, great.  Thanks a lot.  The next Sunday we're in Dunedin, dinner's on me.17:16
gmbintellectronica: How are your test fixes looking? I've got a few more to go, but most of them are just little changes due to bits of the page being moved or restyled, so you can pretty much take a look whenever you like.17:23
intellectronicagmb: exactly the same. how about we both create MPs and exchange reviews at 1800?17:25
gmbintellectronica: Works for me.17:25
=== deryck is now known as deryck[lunch]
gmbintellectronica: I'm going to go grab some food before I review your branch; ping me with the MP link and I'll take a look in half an hour or so. Mine's at https://code.edge.launchpad.net/~gmb/launchpad/bugtask-index-conversion/+merge/12075.17:53
intellectronicagmb: i also have to go away for a bit, to eat and call my family (it's the evening of the jewish new year and they'll take me off the will if i don't call). on the positive side i am in the middle of fixing the last broken test, so i should have the complete fing for you when you come back17:55
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles] || queue: [BjornT, abentley, Edwin, Edwin, sinzui, bigjools]
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [] || queue: [noodles, BjornT, abentley, Edwin, Edwin, sinzui, bigjools]
barryactually: noodles775, BjornT, abentley, EdwinGrubbs, sinzui, bigjools because there are so many reviews on the queue, i'll prioritize to those for folks who are around.  in that order, who's waiting on a review?18:15
abentleybarry: I'm around.18:15
barryabentley: cool.  let's give a minute or so to noodles775 and BjornT and if they don't respond i'll start on yours18:16
BjornTbarry: i'm kind of around, but the review is non-urgent18:17
=== sinzui changed the topic of #launchpad-reviews to: on call: barry || reviewing: [] || queue: [noodles, BjornT, abentley, Edwin, Edwin, bigjools]
barryBjornT: thanks18:18
sinzuibarry: Mine is not important today18:18
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [abentley] || queue: [noodles, BjornT, Edwin, Edwin, bigjools]
barrysinzui: cool18:19
abentleyBjornT: Thanks.18:19
barryabentley: just to be sure, we're talking your restyle-subscriptions branch?18:19
abentleybarry: Yes.18:19
barrycool18:19
EdwinGrubbsbarry: I'm around18:19
=== salgado-lunch is now known as salgado
barryEdwinGrubbs: okay, i'll do yours after i finish abentley's18:24
barryabentley: are there mp's in sample data?18:29
abentleybarry: I don't think so.18:29
barryabentley: ok18:29
barryabentley: ui-wise i have one minor quibble, but i don't think it's introduced by your branch (it shows up in the branch subscribers too)18:34
abentleybarry: oh?18:34
barryabentley: [mail-icon] to all changes: doesn't sit right for me18:35
barryi think i'd like to see the word 'email' in addition to the mail-icon18:35
barryabentley: but if you didn't introduce that, i won't push it18:35
barry(and you might disagree anyway :)18:35
abentleybarry: Not introduced by me.18:35
barryabentley: cool.  i'll mention it but leave it at that.  it's a beuno question anyway18:36
abentleybarry: I figured the icon was meant to represent "email subscribers"18:36
barryabentley: other than that, you have two headings, so that'll need to be fixed18:36
barryabentley: well, actually that's not in your change either18:37
abentleybarry: rockstar is working on a general overhaul of that page.18:38
barryabentley: r=me, ui=me for your branch.  bonus points if you can get rid of that double header on branch merge proposal pages18:38
barryabentley: fab18:38
barrynm then! :)18:38
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [edwin] || queue: [noodles, BjornT, Edwin, bigjools]
barryEdwinGrubbs: which one do you want me to take first?18:41
EdwinGrubbsbarry: it doesn't matter.18:42
barryEdwinGrubbs: cool.  rdf-index-ui-3?18:43
EdwinGrubbssure18:43
=== salgado changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [noodles, BjornT, Edwin, bigjools]
barrysalgado: yay! you fixed the test failures?18:45
salgadobarry, yep, they were all mechanical, so it didn't take thaaat long18:46
=== deryck[lunch] is now known as deryck
barrysalgado: been there :)18:46
salgadobarry, I'll take noodles775's; that ok with you?18:47
barrysalgado: yep, thanks18:49
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [noodles, BjornT, Edwin, bigjools, Edwin]
barryEdwinGrubbs: rdf branch r=me18:58
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [noodles, BjornT, bigjools, Edwin]
EdwinGrubbsbarry: thanks, btw, I added another mp to the list18:58
barryEdwinGrubbs: cool.  i'll do bug 430663 now18:59
mupBug #430663: convert  sprint-register.pt to 3.0 layout <Launchpad Blueprints:In Progress by adeuring> <https://launchpad.net/bugs/430663>18:59
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [noodles, BjornT, bigjools, Edwin, Edwin]
=== henninge changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [noodles, BjornT, bigjools, Edwin, Edwin, henninge (LP 3.0 home page!)]
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [noodles, BjornT, bigjools, Edwin, henninge (LP 3.0 home page!)]
barryEdwinGrubbs: minor comments on bug 430663 branch.  should be easy to19:16
barry        fix.  review sent19:16
mupBug #430663: convert  sprint-register.pt to 3.0 layout <Launchpad Blueprints:In Progress by edwin-grubbs> <https://launchpad.net/bugs/430663>19:16
henningebarry, salgado: I appended my branch with the new LP home page. It's just shy of 800 lines, sorry ;-). I am currently running late for my own birthday party so I hope you are able to approve without asking many questions (in email). ;)19:20
henningeI will be back during the (my) night to land it.19:20
henningebarry, salgado: Please? (I think I forgot that.:)19:23
barryhenninge: happy birthday! 19:23
henningeThank you very very much!19:23
salgadohenninge, I'll try hard to get to it, but all branches today seem to be rather long.19:23
barryhenninge: rs=me if you blow out all the candles with one puff :)19:23
salgadohenninge, and happy birthday, btw! :)19:23
bigjoolsbarry: I iz here, belatedly19:23
henningebarry:  actually, I almost did that yesterday (it was yesterday)19:24
barryhenninge: :)19:24
henningebut they weren't 40 candles .. ;)19:24
barrybigjools: cool.  we're getting to 'em19:24
bigjoolsrockin19:25
barryEdwinGrubbs: another review sent19:39
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [noodles, BjornT, bigjools, henninge (LP 3.0 home page!)]
=== salgado changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [BjornT, bigjools, henninge (LP 3.0 home page!)]
=== salgado changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [edwin] || queue: [bigjools, henninge (LP 3.0 home page!)]
salgadobigjools, do you need a UI review of your trivial-ui branch?19:57
bacsinzui: would you have time to look at https://code.edge.launchpad.net/~bac/launchpad/bug-422334-contact-team/+merge/1208919:58
bigjoolssalgado: probably, it has some stuff other than rs= heading changes19:58
sinzuiye19:58
bigjoolsalbeit trivial19:59
EdwinGrubbsbarry: I can't figure out why the SprintAttendanceRegisterView doesn't have an error when I remove the page_title, but FeaturedProjectsView needs it. Your review comments seemed to anticipate this.20:00
sinzuibac: r=me20:00
bacsinzui: thanks20:00
sinzuibac I had forgotten there was a macro. That was nice clean work20:00
bacthx20:01
barryEdwinGrubbs: it's because FPV is a "top level" view that doesn't have a +hierarchy (i.e. breadcrumb) adapter, so it can't find a default <title>.  SARV has a +hierarchy adapter, thus breadcrumbs, thus default <title>20:01
EdwinGrubbsaha20:02
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [] || queue: [bigjools, henninge (LP 3.0 home page!)]
barryEdwinGrubbs: one more down20:03
salgadobarry, would you mind doing UI reviews for bigjools and henninge and I review the code?20:04
barrysalgado: it's a deal!20:04
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [henninge[ui],-] || queue: [bigjools, henninge (LP 3.0 home page!)]
bigjoolsthanks barry20:18
deryckbarry, salgado -- I've got a branch for review.  Can I get in line? :)20:41
barryderyck: yep, and i can do it after i do bigjools ui review. 20:41
salgadoderyck, sure!20:41
=== deryck changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [henninge[ui],-] || queue: [bigjools, henninge (LP 3.0 home page!), deryck]
deryckcool, thanks, guys.20:42
deryckit's here when one of you is available -- https://code.edge.launchpad.net/~deryck/launchpad/milestone-portlet-links-385719/+merge/1209220:42
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [henninge[ui],-] || queue: [bigjools, henninge (LP 3.0 home page!), deryck, Edwin]
rockstarabentley, ping20:51
abentleyrockstar: pong20:52
EdwinGrubbsbarry, salgado: I have another mp but I'll be afk for about 30 mins. https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-430590-specificationgoal-setgoals.pt-3.0-layout/+merge/1209520:52
barryk20:52
rockstarabentley, can you do one more review for me today?  It's not very big, and very mechanical.20:53
barrybigjools, salgado am i ui-reviewing trivial-ui branch?20:53
abentleyrockstar: okay20:53
bigjoolsbarry: please20:54
barrybigjools: do you have screenshots by any chance?20:54
bigjoolsjust make sure I got the heading stuff right!20:54
bigjoolsbarry: I don't put I put URLs in the MP for you20:54
barrybigjools: k20:54
bigjoolss/put I/but I/20:55
bigjoolsit's late, I need some beer20:55
bigjoolsbarry: there's no major changes, just a few small things lumped into one branch20:57
barrybigjools: cool20:58
barrybigjools: the demo urls say dogfood, but i figured them out21:02
bigjoolsoops21:02
barrybigjools: one of the pages has busted headings, the rest look fine21:02
barrybigjools: https://launchpad.dev/~cprov/+archive/ppa/+packages21:02
barrybigjools: do you need a screen shot?21:02
bigjoolsah I didn't fix that one21:02
bigjoolsI can check here, one mo21:03
barryk21:03
bigjoolsbarry: ok what's bustifuct? :)21:04
barrybigjools: the Packages in "PPA for Celso Providelo" heading shoudl be above the breadcrumbs, not below it21:04
bigjoolsokay21:05
bigjoolseasy21:05
barrybigjools: should be, yeah.  i'll ui approve it with that caveat21:05
bigjoolsbarry: splendid, thanks21:05
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [deryck,-] || queue: [bigjools, henninge (LP 3.0 home page!), deryck, Edwin]
=== salgado changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [deryck,-] || queue: [bigjools, henninge (LP 3.0 home page!), deryck, Edwin, noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
abentleyrockstar: Could you give me a ui review of https://code.edge.launchpad.net/~abentley/launchpad/restyle-subscriptions/+merge/12008 please?21:08
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [deryck,-] || queue: [bigjools, henninge (LP 3.0 home page!), Edwin, noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
bigjoolsbarry: I can't remember how to fix the ugly +blah breadcrumb ...21:10
rockstarabentley, done.21:11
abentleyrockstar: Thanks.21:11
rockstarabentley, got a review coming your way.21:12
barrybigjools: i don't remember if salgado's branch has landed yet or not.  i think you set page_title to the last path component and do /not/ set title_override_breadcrumbs.  salgado can you confirm?21:27
barryderyck: review sent21:27
=== barry changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [-,-] || queue: [bigjools, henninge (LP 3.0 home page!), Edwin, noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
deryckbarry, cool, thanks21:27
bigjoolsbarry: yeah I have page_title implemented but no change to the crumbs.  I guess I'll wait21:27
salgadobarry, bigjools, there's no need to do anything to have that fixed.  it will be fixed once my branch lands.  it's on PQM now21:28
barrysalgado: fantastic21:28
barryokay, i need a detox :)  back in a bit.21:28
bigjoolsossum21:28
deryckbarry, serieses was there before me ;)  I was being lazy. :)21:28
bigjoolsoh and thanks for the code review salgado21:29
salgadobigjools, np. do you have another branch for review or can I remove yours from the queue?21:30
=== salgado changed the topic of #launchpad-reviews to: on call: barry,salgado || reviewing: [-,henninge] || queue: [Edwin, noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
bigjoolssalgado: I only had one, it's not been taken out of the queue21:30
bigjoolsnow it has :)21:31
abentleyrockstar: it looks like the diff includes salgado's work.  Is that right?21:33
rockstarabentley, yeah, because I merged in the middle of it.  I think salgado's work has probably landed in devel by now.21:34
salgadoit hasn't because there's a shipit branch playing on PQM now21:34
rockstarabentley, I tried to go into the future where prerequisite branches are supported in Launchpad, but I got to the 3.0 release and decided it's a future I'd rather not see coming.  :)21:35
abentleyrockstar: It would be nice to diff against salgado's branch, then.21:35
rockstarabentley, coming up.21:36
abentleyrockstar: Your diff also seems to include *my* work-- adding SELECT, INSERT on public.previewdiff21:39
rockstarsalgado, was your branch against db-devel?21:39
salgadorockstar, yep21:40
rockstarabentley, alright, I'm going to re-propose against db-devel.  *groan*21:40
rockstarabentley, re-proposed against db-devel, attached a diff of just my work.21:42
abentleyrockstar: You mean, in addition to the bzr send diff?21:42
rockstarabentley, no, I didn't send this time.  I just created it through the web ui.21:43
abentleyrockstar: What's with pageheading in lib/lp/answers/browser/questiontarget.py ?21:48
rockstarabentley, what do you mean?21:48
rockstarThe original var was name pageheading.  I don't believe it's used anymore.  I'll rip it out and see what breaks.21:48
abentleyrockstar: I'm surprised it's not deleted.21:49
abentleyrockstar: You're frequently doing pageheading = page_title21:49
rockstarabentley, yeah, like I said, I'll delete them, see what breaks.21:49
gary_posterEdwinGrubbs: OK, if you are still willing, MP is up: https://code.edge.launchpad.net/~gary/launchpad/launchpad-templates-3/+merge/12101 .21:51
abentleyrockstar: The i18n Message class is why it says "FAQs for $displayname" ?21:51
rockstarabentley, yeah, because the title is actually a Message, and apparently whatever is generating the title and breadcrumbs doesn't understand that.21:52
rockstarSo it's not making the substitutions correctly.21:52
=== salgado changed the topic of #launchpad-reviews to: on call: barry || reviewing: [-] || queue: [Edwin, noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
abentleyWhat are the high-bit characters in lib/lp/answers/stories/question-answers-vhost.txt?  Culy quotes?21:54
salgadobarry, my head is hurting quite a bit, so I'll stop doing reviews.  I'll do them on Monday (or during the weekend if nobody beats me to them)21:56
abentleyrockstar: ^^21:59
rockstarabentley, I'm not entirely sure, but the original test had the actual characters where I changed them to reprs where I could.21:59
barrysalgado: looks like there's just to more on the queue.  hope you feel better and have a good weekend22:02
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [Edwin] || queue: [noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
abentleyrockstar: Could you please make the imports in lib/lp/registry/browser/person.py that you changed alphabetical?22:04
rockstarabentley, sure.22:04
rockstarabentley, also, it seems that there is no permission check currently on "Edit import or review source"  WTF?22:04
abentleyrockstar: Didn't I point that out in the last review?22:05
rockstarabentley, no, I mean that the current code in devel has no checks either.  Like, no check to make sure you're vcs-imports or anything.22:05
rockstarOdd.22:05
abentleyrockstar: Yeah, I certainly observed that in the last review.22:06
abentleyrockstar: So, do you think pageheading can die?22:07
rockstarabentley, does that mean anyone can review imports now?22:07
rockstarabentley, I think it can.22:07
kfogelAnyone can review a quick and easy one?22:07
kfogelbarry or EdwinGrubbs: easy one: https://code.edge.launchpad.net/~kfogel/launchpad/cc-script-update-reference/+merge/1210222:07
abentleyrockstar: I don't think so.  I think it means that people who can't review imports are still given the link, and then it dies when they actually perform the review.22:08
barrykfogel: if EdwinGrubbs can't do it, but it on the list.  i might have a few more in me today22:08
kfogelbarry, EdwinGrubbs: somewhat urgent, as the community-contributions cron update is broken without this (filing a bug on the larger issue now, but this change contains something that happens to work around the bug coincidentally)22:08
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [Edwin] || queue: [kfogel, noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
rockstarabentley, eep.22:08
barrykfogel: i'll do it when i'm done with this one22:09
kfogelbarry: thank you22:09
abentleyrockstar: Anyhow, I was serious about adding enabled_with_permission, and when you've done that, it'll be good.22:09
rockstarabentley, and edit_whiteboard does need launchpad.AnyPerson as well.  I can't believe we haven't gotten more spam through whiteboards.22:10
rockstarabentley, that was an acknowledgement that I was making the change to edit_import as well.22:10
abentleyrockstar: okay, r=me with the pageheading removed and the imports resorted.22:11
=== salgado is now known as salgado-afk
rockstarabentley, great, thanks so much!22:12
rockstarabentley, shall we have our standup so you can have your weekend?22:12
abentleyrockstar: you bet22:13
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [noodles[ui]] || queue: [kfogel, noodles(https://code.launchpad.net/~michael.nelson/launchpad/409187-trivial-ui-fixes-for-p3a-access/+merge/12035)]
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [kfogel] || queue: []
rockstarbarry, can I have a UI review for https://code.edge.launchpad.net/~rockstar/launchpad/branch-index-redesign/+merge/1206122:34
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on call: barry || reviewing: [kfogel] || queue: [Edwin]
rockstarThere's pictureses and everything.22:34
EdwinGrubbsbarry: I've added another branch to the queue if you have time.22:34
EdwinGrubbsgary_poster: I'm starting on your review now.22:34
gary_posterThank you EdwinGrubbs 22:34
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [kfogel] || queue: [Edwin,rockstar]
rockstarEdwinGrubbs, if you have time, you might be able to do the UI review as well for me, please?22:37
rockstar...or sinzui?22:37
EdwinGrubbsrockstar: I can do it after gary's22:38
rockstarEdwinGrubbs, cool, thanks.22:38
barryEdwinGrubbs: don't do kfogel 's; i just approved it22:39
EdwinGrubbsok22:39
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [Edwin] || queue: [Edwin,rockstar]
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [rockstar] || queue: [Edwin,rockstar]
barryactually, i'll do rockstar's ui review and then EdwinGrubbs's code review22:40
rockstarbarry, fank you.22:40
=== matsubara is now known as matsubara-afk
barryrockstar: when you say "into unittests" another successful strategy we've taken in registry is to move lots of tests into view tests, which can still be doctests22:41
barryrockstar: then actual pagetests are used just for workflow/navigation22:42
rockstarbarry, yeah, that's actually what I'm planning on doing.22:42
barryrockstar: awesome22:42
rockstarbarry, and the vigor to do such a thing is still in me.  We'll see what happens this weekend.22:42
barry:)22:42
barryrockstar: the second screenshot is funky.  is there any way not to wrap that h1 heading with the branch url?22:43
rockstarbarry, not with that extra slot, although I expect that extra slot is in the wrong place.22:44
rockstar(but I don't think that's an issue to be addressed in this branch)22:44
barryrockstar: agreed. can you please file a bug on that?  i'd like to fix that post 3.0.  other than that, your two screenshots like good22:45
rockstarbarry, yay!  Thanks.  I'll file that bug before it lands.22:45
barryrockstar: thanks!22:46
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [edwin] || queue: []
=== abentley1 is now known as abentley
EdwinGrubbsgary_poster: What url is HWDBFingerprintSetView used for?22:54
=== barry changed the topic of #launchpad-reviews to: on call: barry || reviewing: [] || queue: []
=== barry changed the topic of #launchpad-reviews to: on call: - || reviewing: [] || queue: []
barryfolks, that's it for me.  i can't eat any more no matter how wafer-thin22:56
gary_posterI'm not sure--as I said, I was unable to test that.  It is selected in a navigation class.  I'll find it for you22:56
gary_posterEdwinGrubbs: look in lib/canonical/launchpad/browser/hwdb.py .  Look at the HWDBApplicationNavigation class.  Look at the traverse_hwdb_fingerprint method22:57
gary_posterEdwinGrubbs: That's kinda te extent of my knowledge. :-/22:58
gary_posterthe22:58
gary_posterEdwinGrubbs: will be afk for dinner but will check back23:08
EdwinGrubbsthat's fine23:08
EdwinGrubbssinzui: if a page is just used as a macro, it doesn't need a page_title, right? I'm wondering why some exist in pagetitles.py to begin with.23:29
sinzuiEdwinGrubbs: I am sure you have found a mistake23:30
sinzuiEdwinGrubbs: The page must have a @@main_template/master by accident, which means it needs a pagetitle, but if it is never used as a page, that can be removed23:31
EdwinGrubbsrockstar: I'm starting on the ui review of your branch-index-redesign branch.23:58
rockstarEdwinGrubbs, great, thanks.23:58

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