/srv/irclogs.ubuntu.com/2011/12/22/#launchpad-dev.txt

wallyworld_huwshimi: you 100% sure we can't use another colour besides grey for the links? since we use grey for disabled links elsewhere, users will be confused and assume these links are also disabled00:54
huwshimiwallyworld_: Where do we use grey for disabled links?00:55
huwshimiwe use grey for our breadcrumb links00:55
wallyworld_huwshimi: one example, if one enters text for a comment comment or elsewhere, and the text contains "bug xxx", the "bug xxx" is linked and if it is not a valid link, is rendered grey00:56
wallyworld_s/comment comment/bug comment00:56
wallyworld_the breadcrumbs are sort of self contained, not mixed in with general content00:57
wallyworld_but if there's no good alternative.....00:57
huwshimihmmm... I can't reproduce the bug comment links in .dev00:58
wallyworld_a very light blue perhaps? like the tags that are rendered in the bug details page?00:58
wallyworld_ah, ok. you mention light blue looks like disabled00:59
wallyworld_huwshimi: i'll find an example on qas00:59
huwshimiwallyworld_: cheers :)00:59
wallyworld_it is a light shade of grey btw01:00
wallyworld_huwshimi: https://bugs.qastaging.launchpad.net/ubuntu/+source/aptdaemon/+bug/69695401:02
_mup_Bug #696954: Maintainers cannot specify  who is in an access policy <disclosure> <Launchpad itself:Triaged> < https://launchpad.net/bugs/696954 >01:02
wallyworld_huwshimi: i thought i entered a valid bug for one example, but it is also invalid01:02
wallyworld_huwshimi: the valid links are the same shade of blue as the other links01:02
wallyworld_huwshimi: brb01:03
huwshiminp01:03
huwshimiOh I see they only become grey links after a page refresh01:05
wallyworld_huwshimi: yes, there's a background call which does the validity checking. the fact that it doesn't run when a comment is first entered is a bug01:11
huwshimiwallyworld_: Ah I see01:11
wallyworld_which i only just realised happens01:12
* wallyworld_ files a bug01:12
wallyworld_huwshimi: if i recall correctly, there's a css style for disabled links which is used01:12
wallyworld_and the style specifies that shade of grey01:13
huwshimiwallyworld_: I'm inclined to land this with the links this colour and wait to see if there's feedback.01:19
wallyworld_huwshimi: ok. sorry to be a pain rtc. i wanted to be sure we did want to go with those colours given the potential for confusion01:20
huwshimiwallyworld_: No that's ok. I understand where you're coming from, but I think this might be the best option for now01:21
wallyworld_huwshimi: r=me :-)01:23
huwshimiwallyworld_: Cheers mate01:23
wallyworld_np. how was the cricket?01:23
pooliehuwshimi, wallyworld_, re grey links, fwiw01:24
wallyworld_hi poolie01:24
poolie1- other people use them and it's fine; i don't think there is a general style that they have to be blue or underlined anymore (though generally a hover of some kind is good)01:24
poolie2- ideally i think we would be steering towards a new general style (i guess you are) rather than doing it one by one01:25
huwshimiwallyworld_: It was great fun, nice to have a second win too,01:25
wallyworld_poolie: point 2 holds a bit of sway with me. i personally would like to see a consistent colour for disabled across the board, so a user never has to guess if a link is active or not depending on the context01:26
wallyworld_which is sort of what they need to do now01:26
wgranthuwshimi: The bug listings are now arranged pretty strangely: http://people.canonical.com/~wgrant/launchpad/misarranged.png01:29
huwshimiwgrant: Yeah, there's some issues there with consistent column widths01:31
wgranthuwshimi: It also looks like the vertical text alignment in the status/importance labels is off now.01:33
wgrantI thought it used to be centered, but now the text is too high.01:33
huwshimiwgrant: Are you using Firefox?01:34
wgranthuwshimi: Yeah, this is Precise's Firefox.01:34
wgrantSome 9.0 prerelease.01:34
huwshimiwgrant: In Chromium the text is actually too close to the bottom :)01:34
wgrantYay :)01:34
micahg9.0 beta 6 in precise, 10.0 beta 1 will come soon01:36
wgranthuwshimi: Heh, so it is.01:36
huwshimiwgrant: I love web typography01:36
cjwatsonand Tags is about a pixel lower than the bug title text in that screenshot01:36
pooliewallyworld_, i agree, a consistent style for 'disabled' would be nice01:51
pooliei think they can both be grey as long as the intensity is markedly different01:51
wallyworld_which they are01:51
wallyworld_disabled is a lighter shade01:51
wallyworld_poolie: it seems that a feature flag i'm trying to use in a test via "with FeatureFixture(xxxx)" isn't being found when I try and access it from inside a zope security adaptor. have you come across this before?03:14
pooliewallyworld_, ah, i can believe you but i don't have an easy answer sorry03:15
pooliei don't understand the zope magic enormously well03:15
wallyworld_np03:15
pooliesuggest you thrash around until it works, then document that03:15
poolie:}03:15
wallyworld_yeah, i've been thrashing alright. more like drowning03:15
wallyworld_poolie: found it. i am a dickhead. cut'n'paste error. one char difference03:24
* StevenK ponders quoting wallyworld_'s first two sentences out of context.03:33
wallyworld_StevenK: lol03:33
wgrantHm04:49
wgrantI think jtv's megalint and sinzui's apocalypse have conflicted.04:49
StevenKWhat did sinzui apocalypse?04:50
wgrantinterfaces and scripts04:50
wgrantjtv reordered lots of imports.04:50
wgrantSome of which sinzui moved.04:50
wgrantand now the test suite complains on startup.04:50
wgrantLet'04:50
wgrants see how bad it is.04:50
wgrantMay just be the one import.04:51
wgrantAnd fixed.04:55
wgrantHopefully buildbot hasn't finished recently.04:56
StevenKwgrant: It still has 90 minutes.05:11
=== lifeless_ is now known as lifeless
huwshimiOK, that's EOY for me. Have a good Christmas/New Years everyone.06:35
StevenKhuwshimi: You too!06:35
Ursinhacan anyone explain why I can see a bug has 50 activities but trying to index them fails as out of range?07:46
UrsinhaIn [215]: len(lp.bugs[736661].activity)07:47
UrsinhaOut[215]: 5007:47
UrsinhaIn [216]: lp.bugs[736661].activity[0]07:47
UrsinhaIndexError: list index out of range07:48
wgrantUrsinha: Interesting -- it works for me.07:50
wgrantI wonder if you have a broken cache.07:50
wgrantWhich version of launchpadlib are you using?07:50
Ursinhawgrant: interesting thing is that is working for me as well -- problem was reported by someone else07:51
wgrantHah07:51
UrsinhaI wonder if he trying to do that anonymously changes that07:51
Ursinhait shouldn't, right?07:51
wgrantAh.07:52
wgrantIt should, but it often does.07:52
Ursinhahehe07:52
Ursinhawhy?07:52
wgrant'tis a bug.07:52
wgrantAny authenticated user can see them.07:52
wgrantBut anonymous ones cannot.07:52
Ursinharight07:52
wgrantCan you file a bug?07:52
Ursinhathat's reported, I presume07:52
Ursinhaah, sure07:52
Ursinha:)07:52
wgrantNot that I know of.07:52
wgrantBut we have had many similar situations.07:53
UrsinhaI'll file a bug07:53
Ursinhathanks wgrant :)07:53
wgrantnp07:53
=== almaisan-away is now known as al-maisan
bigjoolsmorning all09:14
nigelbMorning bigjools09:14
StevenKbigjools: Did you pick up Portal? :-)09:16
bigjoolsStevenK: no, not sure I will09:18
bigjoolshmmm only 50 attemps to ssh into my server yesterday09:18
bigjoolsspammers are getting around fail2ban09:18
nigelbhow long is your ban time?09:21
nigelbI used to have it as 24 hours :)09:21
bigjoolsdefault, whatever it is09:22
* bigjools is fixing the db-devel conflict10:46
bigjoolsnow why does pqm say only one conflict when locally I get 13210:51
wgrantbigjools: You're probably merging devel, rather than stable?10:55
wgrant(132!?!?!(10:55
wgrant)10:55
bigjoolsnope11:10
bigjoolscan someone else try?11:10
bigjoolsdown to 39 with --lca11:11
bigjoolsok, out of date checkout it seems11:12
bigjoolsgah, 7 ".moved" conflicts11:13
wgrantYeah, deleted dirs do that, because of all the pycs inside :/11:14
wgrantIt'll work better if you have clean trees.11:14
bigjoolsno, it's an added dir11:14
bigjoolsI use a checkout as a sandbox11:14
StevenKwgrant: I'm so tempted to patch rf-get to run 'make clean' before pulling12:06
wgrantThat would be very slow...12:07
bigjoolsand I will shoot you12:36
=== al-maisan is now known as almaisan-away
rick_h__anyone able to review https://code.launchpad.net/~rharding/launchpad/back_sort_buttons_907376/+merge/86701 for me please?13:37
rick_h__backing out fix for bug 907376 so I can refix :/13:37
_mup_Bug #907376: Order By Bar "buttons" do not appear on page load <bug-columns> <qa-bad> <regression> <Launchpad itself:Fix Committed by rharding> < https://launchpad.net/bugs/907376 >13:37
gary_posterIs anyone working on resolving db_devel?  bigjools, you happen to know?  If not I can try to do it before my next call14:18
sinzuigary_poster, I can fix the text conflict if you have not already.14:23
gary_postersinzui, I just did a merge and was about to look at it.  If you are farther along feel free14:23
sinzuiI have done nothing. I expect import conflicts between jtv and the four horseman of the apocalypse14:24
gary_postersinzui :-) ok I'll proceed and ask if I have questions.  thank you14:25
bigjoolsgary_poster: I did it already14:34
gary_posterbigjools, :-P14:34
gary_posterok that wasted time :-) thank you though14:34
bigjoolsgary_poster: however, PQM hated me14:34
bigjoolsargh14:35
bigjoolsgary_poster: did you submit anything yet?14:35
bigjoolsI messed up my commit line14:35
flacostegary_poster, bac, gmb, danilos, benji: lp2kanban got some publicity: http://blog.leankitkanban.com/2011/12/leankit-kanban-api-wrappers/15:34
benjicool15:35
danilosyeah, cool15:38
bigjoolsallenap: I can has review pls? https://code.launchpad.net/~julian-edwards/launchpad/get-incomplete-ppa-copy-jobs/+merge/8672616:04
allenapbigjools: Sure.16:05
bacflacoste: yay16:27
bigjoolsjelmer: mind if I assign you a question about java memory limits when building recipes?17:48
bigjoolsactually I think the java bit is a red herring17:48
jelmerbigjools: can you perhaps link it to a bug?17:49
bigjoolsjelmer: sure, which one?17:49
jelmerbigjools: I suspect it's this one: bug 69352417:49
_mup_Bug #693524: Daily builds of Java packages fail: "Could not reserve enough space for object heap" <recipe> <Launchpad Auto Build System:Triaged> < https://launchpad.net/bugs/693524 >17:49
bigjoolsjelmer: https://answers.launchpad.net/launchpad/+question/182645 is the question17:49
bigjoolsthanks, I'll tell him to follow the bug17:50
* deryck goes offline for lunch, back soon18:26
rick_h__deryck: if you get a sec can you let me know if I'm missing a step with the rollback?19:09
deryckrick_h__, sure, something makes you think you've missed a step?  Or something not working?19:10
rick_h__deryck: well it's just not updating and I ran the  bzr lp-land --rollback=14569 this morning19:10
rick_h__so before EOD wanted to make sure I was waiting on pqm and it's not waiting on me19:10
deryckrick_h__, what's not updating?  the bug?19:10
rick_h__deryck: right http://lpqateam.canonical.com/qa-reports/deployment-stable.html and the bug19:10
rick_h__https://dev.launchpad.net/QA/QAForContinuousRollouts says that the bug should be updated and all qa-* tags removed19:11
deryckrick_h__, has it finished playing in buildnot yet?19:11
rick_h__deryck: ah, problably not. I thought I had skipped that step19:11
rick_h__deryck: but right, I only skipped my ec2 tests, not buildbot19:11
deryckrick_h__, no, you may have skipped ec2, but you still have to pass buildnot to get merged from devel to stable....19:12
deryckrick_h__, once it merges to stable the bug will be updated.19:12
rick_h__deryck: ok, I'll hold tight then. Thanks.19:12
rick_h__deryck: diff topic, is there a good bit of JS code dealing with ajax that's a good template for testing/doing ajax stuff in our "way"?19:12
deryckrick_h__, well any of the order by bar or visible info widget stuff we just worked on is pretty typical.19:13
rick_h__deryck: doh true. Was looking through app/javascript19:14
deryckrick_h__, heh, yeah, np.19:16

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