/srv/irclogs.ubuntu.com/2011/01/13/#launchpad-reviews.txt

=== danilo_ is now known as danilos
wgrantAnyone around for a review?09:11
wgranthttps://code.launchpad.net/~wgrant/launchpad/rebuilds-without-nai/+merge/4607009:11
=== matsubara-afk is now known as matsubara
=== mrevell is now known as mrevell-luncheon
=== mrevell-luncheon is now known as mrevell
=== jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
=== matsubara is now known as matsubara-lunch
=== matsubara-lunch is now known as matsubara
=== deryck is now known as deryck[lunch]
=== benji is now known as benji-lunch
=== gary_poster is now known as gary-lunch
=== deryck[lunch] is now known as deryck
=== benji-lunch is now known as benji
leonardrjcsackett, if you please: https://code.launchpad.net/~leonardr/launchpad/fix-apidoc/+merge/4616117:54
=== jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: leonardr || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
jcsackettleonardr: sure thing.17:55
=== sinzui changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: leonardr || queue: [sinzui] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
jtvjcsackett: can I push one onto your queue?18:24
jcsackettjtv: sure. link to the MP?18:24
=== jtv changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: leonardr || queue: [sinzui, jtv] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
jtvjcsackett: https://code.launchpad.net/~jtv/launchpad/bug-702228/+merge/4616518:25
=== gary-lunch is now known as gary
=== gary is now known as gary_poster
jcsackettleonardr: r=me. i've requested follow up from sinzui.18:36
leonardrjcsackett,great18:36
jcsackettsinzui: i probably should not review your branch, since you're still my mentor.18:36
sinzuijcsackett: I think you should because someone needs to know about mailman oddities.18:37
sinzuiI will find a mentor after your review18:37
jcsackettsinzui: fair enough.18:37
jcsackettsinzui: okay.18:37
=== jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: sinzui || queue: [jtv] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
sinzuileonardr: ping18:41
leonardrsinzui, yo18:43
sinzuileonardr: what is the magic with scalar_value?18:43
sinzuiScalarValue I mean18:43
leonardrsinzui: it's like HostedFile--a special resource type that we don't want to document18:44
sinzuiIs there an example in the live docs?18:44
leonardrno, it doesn't show up. i'm just making 100% sure it doesn't show up in the future18:45
sinzuiokay18:46
StevenKCan we flag it with @hidden or something that makes WADL generation skip it?18:46
sinzuileonardr: r=me. I noted that the copyright needs updating18:46
leonardrStevenK: we want it in the WADL, we just don't want it in the HTML docs18:47
leonardrit's a resource that the end-user never sees18:47
StevenK@undocumented ? :-)18:47
leonardri don't think it's the server side's job to make that kind of distinction18:49
jcsackettsinzui: so MailmanTestCase doesn't actually isolate from the real proxy?18:52
sinzuiI agree with leonardr. We want the xslt public so users can see how we interpret it, but other's may want to make their own interpretation of the data18:53
sinzuijcsackett: it does. I was stumped by the impossible timeouts when I sketched out the test you are reading.18:53
jcsackettsinzui: okay.18:54
sinzuireading XMLRPCRunner.__init__, I realised it already has a reference to the real proxy18:54
leonardrjcsackett: https://code.launchpad.net/~leonardr/launchpadlib/remove-xslt/+merge/4616719:00
jcsackettsinzui: shouldn't the isolation you do in your test be part of the MailmanTestcase setup?19:01
jcsackettsinzui: not asking you to make that change, but if it's something that should be done, perhaps file a bug?19:01
jcsacketti acknowledge i don't have a full understanding of what we want in a mailman test, but unplugging from the proxy seems correct.19:01
sinzuijcsackett: which setup. I think the test setup id DRY19:04
jcsackettsinzui: nevermind, i was misunderstanding what was being done.19:05
jcsackettsinzui: i thought that you were patching around something the base class for your test should have done. that is not the case.19:05
sinzuiyeah, get_mailing_list_api_test_proxy() looks like duplication, but it is changing an instance, where as super() is changing the module19:06
sinzuireset_log and get_mark may move to the base class. I am pondering a test in a new branch that may need them19:06
sinzuijcsackett: when you are done this my branch, I want to to talk about the implementation of my current bug, the one you looked at this morning19:08
jcsackettsinzui: sure.19:08
jcsackettsinzui: r=me, with two minor things.19:13
jcsackettnotes are on on your diff, but it's basically some docs/comments stuff.19:13
jcsackettleonardr: your MP shows zero lines diff.19:14
leonardrhmm...19:14
sinzuijcsackett: I will follow up on you comments19:14
jcsackettsinzui: sounds good. feel free to ping me when you want to mumble re: your current branch.19:15
jcsackettleonardr: the file you wanted to remove (as i understand it) is still in your branch. http://bazaar.launchpad.net/%7Eleonardr/launchpadlib/remove-xslt/files/head%3A/src/launchpadlib/19:15
=== jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: jtv || queue: [leonardr] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
leonardrjcsackett: i didn't commit anything. new version is pushed19:16
sinzuijcsackett: can we mumble for a few minutes?19:19
sinzuijcsackett: http://pastebin.ubuntu.com/553725/19:28
pcjc2Can someone take a look at https://code.launchpad.net/~pcjc2/launchpad/fix-tag-search-bug-501945/+merge/46075 for me?19:56
=== leonardr is now known as leonardr-afk
=== leonardr-afk is now known as leonardr
=== jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: jtv || queue: [leonardr, pcjc2] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
jcsackettpcjc2: you're on the queue.20:20
jcsackettpcjc2: you're a new contributor?20:20
jcsackettjtv: r=me, but it looks lifeless hit it a bit earlier. i won't request follow up from sinzui since lifeless already approved it.20:39
jcsackettleonardr: r=me.20:39
pcjc2newish - this is my second patch20:39
=== jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: pcjc2 || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
jcsackettpcjc2: cool. so, in the channel info here there is frequently an on call reviewer, who you can usually ping to get into the queue.20:40
jcsackettin this instance, that's me. :-)20:40
pcjc2I hashed out some of this with lifeless (He got the SQL tested), but he's a bit busy at the moment to review it20:40
pcjc2Thanks - I've not been here before20:41
pcjc2The other patch I did was with gmb helping, and it related to bug import, not an actual part of the web service.20:41
* pcjc2 approves of Peter B's config API20:58
pcjc2Might want to consider the possibility of freeze and thaw for the callbacks20:58
pcjc2Also, possibly add save APIs?20:58
pcjc2export either a given context, or a "flattened" (borrowing from gimp layers) context ?20:59
pcjc2(OOPS: -E_WRONG_WINDOW)21:00
sinzuiEdwinGrubbs: thumper: can either of you mentor jcsackett review of my branch? https://code.launchpad.net/~sinzui/launchpad/mailman-heartbeat-0/+merge/4616021:09
EdwinGrubbs sinzui: I can do it21:09
=== jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
jcsackettpcjc2: there are comments on your MP.21:23
pcjc2thanks21:24
* pcjc2 checsk21:24
EdwinGrubbssinzui: it seems strange to log "--MARK--" since syslogd can be configured to add that to a logfile automatically. Of course, when the xmlrpcrunner does it, it would be preceded by a timestamp.21:24
pcjc2"You should flip those back; by convention we put the expected value first in assertEqual statements."21:25
pcjc2correct fix is then to swap the parameters passed to each caller then21:25
pcjc2(and swap the assert back)21:25
sinzuiEdwinGrubbs: I was following the suggestion of the losa.21:25
* sinzui looks at mailman's own logging lib again21:25
jcsackettpcjc2: that may be correct, my comment was limited to the scope of your MP.21:25
pcjc2I swapped them because every test had the parameters reversed, I didn't know there was a convention21:26
sinzuiEdwinGrubbs: mailman does not use python batteries. I think it still thinks we live with Python 2.121:26
pcjc2RE: comment - a docstring at the start of the function ok to explain the SQL?21:26
jcsackettpcjc2: that should work, yes. please remember that the first line of a docstring should be one line quick explanation of the method, then a blank line, then the longer details.21:27
jcsackettand thanks for making these changes. :-)21:27
pcjc2like a git commit ;). NP21:27
jcsackettexactly like a git commit. :-)21:28
jcsackettpcjc2: ping me when you have pushed the changes, i can approve the MP and alert my mentor to follow up.21:28
jcsacketteven if i'm in the middle of another review i can do that quickly.21:29
pcjc2ok, will do. Am merging some git conflicts another project right now, but should look at it tonight21:29
jcsackettpcjc2: ok.21:30
EdwinGrubbssinzui: I just checked, and rsyslog puts the timestamp in front of its autogenerated "-- MARK --", so I think it is a bad idea to use that as your heartbeat text. If the losas insist on that, I'll agree to it, but I'm guessing they didn't think about the confusion it could cause.21:36
sinzuiEdwinGrubbs: I think we want a timestamp so that nagios nows when the heatbeat took place21:37
* StevenK hands sinzui a 'k'21:37
EdwinGrubbssinzui: yes, but if nagios is grepping for "-- MARK --" with a timestamp, and syslogd is configured to add "-- MARK --" entries every few minutes, and they also have a timestamp, the xmlrpcrunner could be broken and you would never know it.21:38
sinzuiI am getting rid of silent letters just after I get rid of c, j, q, x21:38
StevenKYou can't QQ without q21:39
sinzuiEdwinGrubbs: I do not know what syslogd has to do with this. It is not managing Mailman's logs21:40
sinzuiWhy would the losas request that we add --MARK-- to the queue's log if they could do it themselves21:40
sinzuiq is a dipthong for kw21:41
lifelessI think asking a losa is a good idea at this point :)21:43
=== matsubara is now known as matsubara-afk
sinzuilifeless: Mailman's Syslog puts a write lock on the log. As my test shows, some insider knowedege and access is needed to acquire access,21:45
EdwinGrubbssinzui: what is on the receiving end of the syslog() call in the xmlrpcrunner then? Configuring syslogd to log MARK just shows that syslog is working, it doesn't show that the programs using that logfile are working.21:45
sinzuiAnd be assured I toppled mailman several times learning the art of stealing the logs21:46
lifelesswheeee21:46
lifelessEdwinGrubbs: I think the point is that we don't know if xmlrpcrunner is hung or not21:47
sinzuiEdwin Mailman.logging.syslog is a dict-like object that manages Mailman.logging.StampedLoggers. Each logger manages a file with 'a+b'21:47
sinzuiEdwin each object in the chain ensures the file is closed if you were to call del on the instance21:48
sinzuiEdwinGrubbs: Mailman really thinks about logging like it was the 10 years ago. It does not even have bools21:49
EdwinGrubbssinzui: ok, that is bizarro21:50
sinzuiBut it would life to use with `open(ccc) as log:` Mailman 3 might do that21:50
sinzuiEdwinGrubbs: I believe mailman requires Python2.1 to run21:50
sinzuiEdwinGrubbs: I was looking for barry to asking about SHUNTING. I will bring up the log issue with him21:52
sinzuiwhen I find him21:52
EdwinGrubbssinzui: you can keep using MARK if you want, since mailman's syslog() doesn't log to syslogd. This reminds me of the time a friend created a readFoo() function that had a parameter to turn on writing.21:58
EdwinGrubbsI'll approve the mp21:58
* sinzui is still looking for barry21:58
sinzuiEdwinGrubbs: thanks. I really want to talk to barry. I an reading an old log and I makes me wonder if his and mthaddon's analysis is complete. I see log entries that imply my change will work by accident, in some cases22:00
StevenKsinzui: I can find Barry in Dallas if you wish22:03
StevenK(And tempt him onto IRC)22:03
sinzuiplease do22:03
StevenKsinzui: Aye22:03
barrysinzui: be with you in a few minutes22:05
sinzuifab22:05
StevenK\o/22:05
jtvjcsackett: thanks22:05
barrysinzui, okay, i'm all yours22:09
sinzuiLooking at a mailman error log. I see that messages were shunted because of22:10
sinzuiXMLROC proxy errors in LPHandler. I do not think I want the message shunted22:10
sinzuisince Runner implies IncomingRunner or LPHandler should have done something22:10
sinzuiitself.22:10
sinzuiI will wrap the proxy call to catch and raise oopses when xmlrpc goes down, but I think I want the message reenqueued22:10
barryi can't think of a good way to prevent Runner._oneloop() from shunting when _onefile() throws an exception22:12
barryshort of hacking the code, or catching the exception in _onefile()22:12
sinzuiyuck22:12
barrywell, actually, maybe you can set self._shunt to the retry queue22:13
sinzuiI hoped for a way to reenque from the handler, or raise a specific error that tell IncomingRunner to do it22:13
* barry thinks22:13
sinzuiHow will a shunted message ever be processed. No one manages mailman22:14
barryit has to be done manually via bin/unshunt22:14
* barry shudders to think how big the shunt queue is22:14
sinzuiI think then we have several years in there22:14
barryyou almost certainly don't want to unshunt the whole queue ;)22:14
StevenKHow can we see how large the queue is?22:15
thumperbarry: hi!22:15
sinzuiSo as I was thinking. We just need to pause the queue until connectivity is back. just loop over the message22:15
barrythumper, hi!22:15
barryStevenK, ls queue/shunt22:15
barryer, queues/shunt22:15
barrynote tho that the queue files have a timestamp encoded in the file names so you could potentially decode that and only unshunt the ones from whatever date you care about22:16
barrynot sure what you mean by "until connectivity is back"22:17
sinzuibarry, what would you do if you LPHandler has no connectivity to complete its task. this is like a lock on a list. I see we try to return 1 to imply try again later22:17
barrysinzui, right.  just leave the message in the queue and it will be picked up the next time 'round22:18
barryor, enqueue it to queues/retry, but all that does is requeue the message every so often22:18
barryso it's about equivalent22:18
sinzuibarry: Sometimes the xmlrpc request times out, but the next try works. But on Dec 1, it went down and mailman stopped trying.22:19
barrysinzui, any idea why it stopped trying?22:19
barryanother idea...22:20
sinzuibarry: Log looks like Mailman got board shunting22:20
sinzuirestart worked fine22:20
barry;)22:20
barryso, the .pck files have two objects in them: a dict (the message metadata) and the message object22:20
sinzuiMy branch to add a heartbeat will warn us in the future22:20
sinzuibarry: how would I signal/do an enqueue from a handler22:21
barryit would be a bit slow but you can unpickle the .pck file and look in the dict for tell-tale signs that LPHandler is where the failure occurred22:21
sinzuibarry: I can see where the failure happen. There are only two calls in out monkeypatches that do not try/except arround the proxy. I am sure we can log this. I just want solve how to keep the message alive22:22
barrysinzui, you just instantiate the queue you want and call .enqueue() on it.  note tho that that makes a copy of the message, so absent anything else, it continues to get processed by the current runner22:23
sinzuibarry: from inside the handers except clause? That sounds like recursion22:23
barrysinzui, so i think you catch the exception, instantiate the retry queue, enqueue the message, then reraise the exception (or raise DiscardMessage)22:24
barryjust be sure we're running the retry runner (i don't remember and i don't have the lp code in front of me)22:24
barryStevenK, do you have the lp code handy?22:24
StevenKbarry: Yes22:25
sinzuiahh, good point. If I raise the exception, it get shunted. raising discard just removes the copy in in the queue?22:25
barrysinzui, yep22:25
barrylook at IncomingRunner.py22:25
StevenKbarry: You'll need to come to me, my battery is 4% or so22:25
barryStevenK, what room are you in?22:25
sinzuibarry: I know how to check that and I know how to make it run22:25
StevenKbarry: Desktop22:25
barryStevenK,sec...22:25
sinzuibarry: we are running the retryrunner. I wrote a test to ensure it is. Thanks I really appreciate your help22:29
barrynp.  looking at the code now22:29
barryit's been a while ;)22:29
barrysinzui, which call is failing?22:30
barryproxy = proxy = ? ;)22:30
sinzui  File "/srv/lists.launchpad.net/production/launchpad-rev-9972/lib/lp/services/mailman/monkeypatches/lphandler.py", line 49, in process22:30
sinzui    is_member = proxy.isRegisteredInLaunchpad(sender)22:30
sinzuibarry: there that is one of two places we call proxy that we do not wrap in try/except22:31
barrysinzui, yeah, for some reason i think the preceding comment is a lie ;)22:31
sinzuibarry: I was going to reuse the except rules an log_exception from xmlrpcrunner on it22:31
* barry thinks22:32
sinzuibarry: This is one of many entries in the log http://pastebin.ubuntu.com/553793/22:32
barryyeah, it's not a lie because xmlrpcrunner isn't incoming runner22:33
sinzuiThis line and the one in lpstanding is just an oversight. All other calls are safe to use, but this one cold lose the message22:35
* pcjc2 struggles to think of concise words to describe the SQL statements reasoning in his docstrings22:43
barrysinzui, so yeah, i think the best you can do is to catch the exception, enqueue the message to retry, and raise DiscardMessage.  then the retry runner should requeue it, and when it's re-processed by incoming runner it should start back up at the LaunchpadMembers handler22:44
barrysinzui, write some tests, but i think that should work ;)22:44
jcsackettpcjc2: "Because the final clause only checks the existence of results of the query, subqueries can select true rather than an actual entity, speeding up the process."22:49
jcsackettmaybe.22:49
jcsackettit's a little clunky. :-P22:49
pcjc2I'm moving the EXISTS keyword into the build_tag_set_query22:49
pcjc2function22:49
pcjc2then it returns a TRUE or FALSE, and it is cleaner - as well as easier to explain22:49
pcjc2might not even NEED to explain the SQL in detail if that change is made?22:50
pcjc2http://pastebin.com/EfHtBNdE22:52
pcjc2?22:52
pcjc2THink I will drop the comment about where it is used22:52
pcjc2that could get out of date fast22:52
pcjc2Will add one about expecting the surrounding SQL to ensure "Bug.id" is in scope22:52
pcjc2what about http://pastebin.com/QswKQ2Lg ?22:55
pcjc2See the diff: http://pastebin.com/9iViJh6Y23:00
* pcjc2 re-runs tests to be sure23:00
=== jcsackett changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews
jcsackettpcjc2: i would put quick comments above the string format statements, since you end up with things like "%s %s %s".23:06
pcjc2comment to what effect?23:06
pcjc2at some point, one has to read the context of the code ;)23:06
jcsackettpcjc2: that's an excellent question, actually.23:07
jcsackettyeah, this is just an inherently complicated bit.23:07
jcsackettlooks weird, but you're right, it doesn't need anything more.23:07
jcsacketttest passing?23:07
pcjc2"%s %s NOT %s" is a bit strange, I conceed23:07
pcjc2give it a month or two to run...23:07
jcsackettyeah, it's odd, but i think it's alright.23:07
jcsackettlol.23:07
pcjc2(just wanted to check the "testbug.py" test, as it compares SQL output - had to make sure I'd not shifted the way the SQL was braketed23:08
pcjc2I'm not going to re-run the other tests as the SQL is the same..23:08
* jcsackett nods.23:09
pcjc2Will need some manual QA though, as I'm not sure how good the "life" test coverage is23:10
jcsackett?23:11
jcsackettwhich test are you referring to by "life" test?23:11
pcjc2"live" I meant23:11
pcjc2with a big data-set23:11
pcjc2it aims to fix a problem which needs a production sized DB to reproduce23:11
pcjc2       return "%s" % include_clause23:13
lifelesspcjc2: we qa all such things ;)23:13
pcjc2should become "return include_clause"23:13
lifelesspcjc2: on qastaging.launchpad.net23:13
lifelessperhaps23:13
lifelessreturn str(include_clause)23:13
pcjc2include_clause is assembled from strings23:14
lifelessor even unicode(include_clause)23:14
pcjc2we don't prefix the return "(%s %s NOT %s)" % ... case with u"23:14
lifelesspcjc2: sure, explicit can be good sometimes (also its a noop if the type matches)23:14
thumpersinzui: https://code.launchpad.net/~thumper/launchpad/daily-build-page-text/+merge/46207 for your UI approval :)23:15
sinzuiokay23:15
pcjc2lifeless - name your preference... unicode, str ?23:15
pcjc2meh - actually, it is only the one "NOP"  return "%s" % include_clause23:16
pcjc2the other cases around it need that syntax, so it is best we keep them consistent23:16
pcjc2even if "%s" % ...  is  a NOOP23:16
pcjc2what is the SQL'esque way of fixing this comment...23:17
pcjc2This SQL is designed to be a sub-query where the parent SQL defines Bug.id.23:17
pcjc2"defines" seems wrong23:17
pcjc2JOINs/includes/contains/?23:18
jcsackettpcjc2: i have to run for the evening. i'll check the MP in the morning for the changes. ttyl.23:18
pcjc2"parent query scope includes Bug.id" ?23:18
pcjc2@jcsackett: Thanks for the review - with luck the MP will be in good order by tomorrow23:19
jcsackettpcjc2: sounds good.23:19
lifelesspcjc2: I would leave it as is23:19
pcjc2as it was?23:19
lifelesspcjc2: it looked ok at a glance23:19
lifelessas was, yes23:20
pcjc2ok, will push that diff23:20
lifelessjcsackett's two comments were good suggestions23:20
pcjc2Do I need to mark the MP as needing further review?23:20
pcjc2Review comments have been taken into account, new push coming...23:20
huwshimisinzui: Hi. Would you be able to review a CSS bug fix sometime (bug #684911: https://code.launchpad.net/~huwshimi/launchpad/broken-calendar-684911/+merge/46208)?23:26
_mup_Bug #684911: calendar overlay widget style broken after lazr-js 191 upgrade <javascript> <lazr-js-upgrade> <lp-web> <regression> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/684911 >23:27
* sinzui hates that widget23:27
sinzuiyou cruel man.23:27
huwshimisinzui: Is that a no then? :P23:27
sinzuiah, so the hack to for the yui-3 skin was lost in the last update?23:28
huwshimisinzui: Looks like something like that. In which case, me fixing the styles this way would only be a temporary fix23:29
huwshimiI don't really understand how all the lazr-js stuff works yet but maybe the real fix is something to do with that23:29
sinzuiWe wanted a yui-3 widget, but one does not exist. We could make a lazr one that handles NOW23:30
sinzuithis is good to land. Thanks huw.23:30
huwshimiYeah ok that makes sense. Thanks23:31
sinzuiOh, the fonts are wrong23:32
huwshimiAh yes23:32
sinzuihuwshimi: I will find the correct font-family string23:32
sinzuiI remembered that we had to hack the fonts as well as the skin 18 months ago23:32
pcjc2@jcsackett: My MP should be ready to go when you get to it tomorrow - thanks23:34
sinzuihuw: do you have a strong understand of how browsers (last 5 year) handle font-size. YUI requires us to use % to guarantee consistent rendering, but that also means the engineer never knows what the actual font-size will be when all content and rules are mixed together23:36
sinzuihuw I want to switch back to ems, or even px as suggested by Canonical web guidelines if I could trust consistent sizes23:37
huwshimisinzui: Yeah font sizing gets very confusing very quickly.23:43
huwshimiEspecially when you won't necessarily have control over third party widgets etc.23:43
sinzuiI used to avoid px. I was surprised the Canonical insists on it. I think they may be looking a how browser engines handle px though. My phone shows me pages and I know it is ignoring the px rules23:44
huwshimiWith the calendar I can either set the font to the Ubuntu one or remove the font-family settings altogether (which will then inherit the correct fonts)23:45
huwshimiAny preference23:45
huwshimi?23:45
sinzuiYUI is specifically reconciling Window with every other os/basefont. I am not sure I care about windows23:45
lifelesssadly we care23:45
lifelessbecause our customers have customers and suppliers on windows23:45
sinzuihuwshimi: I think we need to set it. Somewhere in the chain, verdana or tahoma gets added23:46
sinzuilifeless: I think the issue windows fonts being a little big is less than the issue that we are not using the font-sizes that the CSS predicts will rendered. % rules are being nested. We cannot avoid that23:47
huwshimisinzui: On a quick test of removing all font-family lines from calendar.css it seems to be working correctly.23:47
sinzuilifeless: http://curtis.hovey.name/2010/12/17/launchpad-font-size-broken-by-design/23:48
sinzuihuwshimi: fab23:48
huwshimisinzui: Should I go ahead and commit without font-family rules then?23:49
sinzuihuwshimi: yep23:49
huwshimiok great thanks.23:49
lifelesssinzui: I can see the brain damage on linux too ;)23:50
lifelesssinzui: would love it if we fix this.23:51
sinzuilifeless: I am hoping for a short brain storm session + a spike at the thunderdome to find a compromise that will let me close a UI bug23:52
huwshimisinzui: When I try and land the branch it complains that the branch is not approved. On the MP the status is approved. Do I need to do anything else to get it to land?23:59

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