/srv/irclogs.ubuntu.com/2011/03/17/#launchpad-yellow.txt

gary_posterhiya.  getting breakfast for kids.  bac, got your message.  I'll be in touch in a few to touch base on what you might want me to work on today.  On your list I was thinking about looking at the flicker, though I might not be able to pull it off quickly.12:01
bacgary_poster: ok12:07
gary_posterhttps://dev.launchpad.net/TestingJavaScript12:42
gary_posterhttps://dev.launchpad.net/Windmill12:43
bachttps://dev.launchpad.net/JavascriptUnitTesting12:43
gary_posterhi gmb and benji.  I see you in mumble already.  I'll be ready in a min.13:29
benjigary_poster: a slightly expanded version of my comment from the call: do you know if the representation for subscribing to all (e.g.) statuses is different than just enumerating all the (current) statuses?  If so we should be sure to use it instead of just including all the statuses in the list.13:58
gary_posterbenji, ah!14:04
gary_posterso, an empty list of statuses/importances means matching all importances14:04
gary_posterthat's what my card is about14:04
gary_poster(matching all status/importances respectively, I should have said)14:04
gary_posterIMO this is what we should do:14:05
gary_poster1) on the server side, if we are told that we are supposed to subscribe to all statuses/importances, we convert that to an absence of statuses/importances in order to keep database records low14:06
gary_poster(each status/importance checkmark per filter == a row in a table)14:06
gary_poster2) in our JS for the form overlay, convert an absence of checkmarks to all checkmarks; I think we might do that already, not sure14:07
gary_poster3) in our JS for summarizing, I think the changes above will make the display "just work" but we should make sure that the summaries do not list all importances/statuses--if they are all selected, then that means there is no filtering on them.14:08
gary_poster(done)14:08
benjiI don't think 2 is neccesary, if they haven't checked anything then they'll get an empty list which means everything (right?)14:11
benji3) it does list every one if they are all present at the moment, but if we do 1 then that won't happen14:12
gary_posterI think it is much clearer to the end user that all checkboxes means that they get all statuses/importances14:12
gary_posterThat's the way it was mocked up for user testing, and that's the way it works in our cyrrent add form14:13
benjioh, you meant in the UI for 2, yeah, that makes sense; I was talking about the API14:13
gary_poster3: right, that's what I was thinking14:13
gary_poster2: oh, cool, we're on the same page then14:13
benjii.e., if you check all of them in the UI the API call should include none (meaning all)14:13
benji1) may be a challenge, I don't know if there's a place we can add that logic14:14
gary_poster2: I don't care about us sending "the right thing" over the API actually, because 1) should make it work the way we want anyway.  That's good because we want people other than ourselves to not create unnecessary records too.14:15
gary_poster#1 can go in the model logic.  It's all already done with properties so it should be pretty straightforward.14:16
benjicool14:16
gmbgary_poster: I'm ready whenever you are.14:30
gary_postergmb, setting duration to 0 appears to set it to some default speed, rather than "immediate" :-/14:56
gmbBoo.14:57
gary_posteragreed14:57
gary_posterok, I'm off to "lunch" (helping out wife with stuff out in the car).  Back in a bit.15:00
benjiI keep getting: WindmillTestClientException: {u'suite_name': u'RegistryYUIUnitTests', u'result': False, u'starttime': u'2011-2-17T20:44:18.244Z...15:16
benjiI've seen it before but rerunning the tests made it go away.  I've gotten it three times in a row now.15:16
* gmb -> errands; bbiab15:21
=== Ursinha is now known as Ursinha-lunch
gary_posterbac, ready for call when you are.  When there, I have this question for you (for reminding myself): can you explain what node.set(SRC, EXPANDER_COLLAPSED); does?18:31
bacok.  1 minute, let me beverage-up18:31
gary_postercool18:32
benjibac: do the windmill tests currently pass for you?19:10
bacbenji: i haven't tried in a while.  OTP but will try later19:11
benjithanks; they consistently fail for me, but I think I'm homing in on the problem19:11
gary_posterhttps://bugs.launchpad.dev/firefox/+subscriptions19:33
gary_posterhttps://bugs.launchpad.dev/firefox/+bug/1/+subscriptions19:33
_mup_Bug #1: Microsoft has a majority market share <iso-testing> <ubuntu> <Clubdistro:Confirmed> <Computer Science Ubuntu:Invalid by compscibuntu-bugs> <EasyPeasy Overview:Invalid by ramvi> <GNOME Screensaver:Won't Fix> <Ichthux:Invalid by raphink> <JAK LINUX:Invalid> <The Linux OS Project:In Progress> <metacity:In Progress> <OpenOffice:In Progress by lh-maviya> <Tabuntu:Invalid by tinarussell> <Tivion:Invalid by shakaran> <Tv-Player:New> <Ubunt19:33
=== Ursinha-lunch is now known as Ursinha
bachi gary_poster20:41
gary_posterhey bac.  otp20:42
bacgary_poster: ok.  can you ping me when you're off?  i'd like a quick chat20:42
gary_postersure20:42
bacgary_poster: any idea how long?20:42
gary_posterbac, 5?20:42
bacgreat20:42
bacer, 5:00pm or 5 minutes?20:43
gary_posterminutes20:43
gary_posterbac, mumble, skype, ...?20:54
bacmumble20:54
benjigary_poster: I'm almost ready to land the test fixes.  It will have one odd bit, I removed one set of tests that the branch I'll land later replaces.  I tried to fix them for the time being but it took too long to warrent putting into tests I was going to rework anyway.21:30
benjigary_poster: ok, finally pushed21:52
gary_posterawesome, thanks benji.  I'll try finish getting the help pages sorted out, and then try to merge, test, & push.22:28

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