StevenK | wgrant: margin:auto doesn't seem like a magic bullet. text-align:center on the container div and display:inline-block on the calender div just results in the widget looking squashed and still on the left of the overlay | 03:07 |
---|---|---|
=== almaisan-away is now known as al-maisan | ||
StevenK | wallyworld: ^ Can you give me some clues about the above? | 04:01 |
wallyworld | StevenK: sadly my css foo is not that good | 04:02 |
wallyworld | curtis is your best bet | 04:02 |
StevenK | wallyworld: Fair enough, I might hit up a few friends of mine | 04:04 |
wallyworld | sorry | 04:04 |
nigelb | StevenK: Is whatever you're trying to fix, online? | 04:06 |
nigelb | I mean, can I see it? | 04:07 |
StevenK | Not really | 04:07 |
StevenK | It's a LP branch, but it requires you have a dev environment | 04:07 |
nigelb | argh. Mine's broken for now :/ | 04:08 |
=== al-maisan is now known as almaisan-away | ||
wgrant | StevenK: Are your latest changes pushed? | 04:40 |
StevenK | wgrant: Nope | 04:40 |
wgrant | Please do so :) | 04:40 |
StevenK | wgrant: I have it working now anyway | 04:40 |
wgrant | I will poke | 04:40 |
wgrant | Ah, good | 04:40 |
wgrant | What'd you need to change? | 04:40 |
StevenK | I needed to apply 'margin: 0 auto' to .yui3-calendarbase | 04:41 |
wgrant | Ah | 04:42 |
wgrant | Screenshot? | 04:42 |
wgrant | What remains to be fixed? Wiring up the selection events properly? | 04:42 |
wgrant | Which is not CSS, so it's trivial :) | 04:42 |
StevenK | No, done | 04:43 |
StevenK | wgrant: http://wedontsleep.org/~steven/new-calendar-4.jpg | 04:44 |
wgrant | StevenK: So it writes the correct format to the textbox, and only when the tick is clicked? | 04:45 |
StevenK | wgrant: Yeah, it will only close the overlay when you click the tick | 04:46 |
StevenK | In the case of showing time as well | 04:46 |
wgrant | Right. | 04:46 |
StevenK | wgrant: If it looks good, I can push it up | 04:46 |
wgrant | StevenK: Sounds good | 04:47 |
wgrant | StevenK: Were you going to integrate that CSS? | 05:06 |
wgrant | Into combo.css | 05:06 |
StevenK | wgrant: Well, yeah | 05:07 |
StevenK | It gets pulled into combo.css as part of combine-css | 05:08 |
wgrant | StevenK: Then why'd is referenced in launchpad-widget-macros.pt? | 05:08 |
StevenK | Oh | 05:09 |
StevenK | Right | 05:09 |
StevenK | Sorry, I thought you meant the other bits I changed in css/modifiers.css | 05:10 |
wgrant | Ah, no | 05:10 |
StevenK | wgrant: So, I can. Not a great deal of pages use the calendar | 05:10 |
wgrant | StevenK: Sure, but it's pretty small and it sucks to have the four extra requests like that. | 05:11 |
wgrant | StevenK: Also, you've split the .inline and .hidden rules, but the order between them matters, so they should ideally always be together. | 05:11 |
wgrant | And those skin overrides will want a comment detailing their motivation. | 05:12 |
wgrant | StevenK: Do you also want to destroy the overlay and calendar when the overlay is hidden? | 05:19 |
wgrant | At present you only destroy them on cancel | 05:19 |
StevenK | Hmm, I should refactor that out | 05:20 |
* StevenK stabs combine-css for not including the calendar css | 05:28 | |
StevenK | combine-css really dislikes yui/assets/skins/sam/calendar-base.css | 05:33 |
wgrant | Howso? | 05:34 |
StevenK | wgrant: http://pastebin.ubuntu.com/5591677/ | 05:34 |
wgrant | StevenK: What if you include the -core and -skin separately? | 05:36 |
wgrant | It may not like the pre-minfied version. | 05:37 |
StevenK | combine-css really dislikes yui/assets/skins/sam/calendar-base.css rather than yui/assets/skins/sam/calendar-base.css works | 05:41 |
StevenK | Bleh | 05:41 |
StevenK | Including yui/calendar/assets/calendar-core.css, yui/calendar/assets/skins/sam/calendar.css and yui/calendar/assets/skins/sam/calendar-skin.css rather than yui/assets/skins/sam/calendar-base.css works | 05:41 |
wgrant | StevenK: Right, that's what I meant. | 05:44 |
wgrant | I wonder if it's a bug in YUI's minification, or a bug in cssutils. | 05:45 |
StevenK | Except I need calendar-base too | 05:45 |
wgrant | Oh | 05:45 |
wgrant | Misread | 05:45 |
StevenK | xml.dom.SyntaxErr: PropertyValue: No match: ('CHAR', u':', 2, 588) | 05:45 |
wgrant | That calendar.css should be redundant with calendar-skin.css and calendar-core.css | 05:45 |
wgrant | foo-core.css + foo-skin.css = foo.css | 05:46 |
wgrant | So you want to include calendar.css, calendar-base-core.css, calendar-base.skin.css | 05:46 |
StevenK | wgrant: http://pastebin.ubuntu.com/5591691/ | 05:46 |
wgrant | Try that | 05:46 |
wgrant | + 'yui/calendar/assets/calendar-core.css', | 05:46 |
wgrant | + 'yui/calendar/assets/skins/sam/calendar.css', | 05:46 |
wgrant | + 'yui/calendar/assets/skins/sam/calendar-skin.css', | 05:46 |
wgrant | + 'yui/assets/skins/sam/calendar.css', | 05:46 |
wgrant | 2 and 4 are the same | 05:46 |
wgrant | And 2 and 4 both include 1 and 3 | 05:47 |
wgrant | calendar and calendar-base are separate components | 05:47 |
wgrant | They each have -skin.css, -core.css and .css files | 05:48 |
wgrant | The .css file (without -skin or -core) is the combination of -skin and -core. | 05:48 |
wgrant | Minified. | 05:48 |
StevenK | wgrant: However, now combine-css croaks, so one of the unminified files causes the same thing | 05:48 |
wgrant | StevenK: Hopefully you can narrow it down. | 05:49 |
StevenK | Just doing so | 05:49 |
StevenK | Both yui/calendar-base/assets/skins/sam/calendar-base.css and yui/calendar-base/assets/skins/sam/calendar-base-skin.css | 05:50 |
wgrant | StevenK: Remember that one of those files is a superset of the other. | 05:52 |
StevenK | Oh, -skin is unneeded | 05:52 |
wgrant | No | 05:53 |
wgrant | -skin is the source | 05:53 |
wgrant | calendar-base-skin.css is a subset of calendar-base.css | 05:53 |
wgrant | So if you want to continue isolating, the next step is to drop to just the -skin | 05:53 |
StevenK | xml.dom.SyntaxErr: PropertyValue: No match: ('CHAR', u':', 12, 19) | 05:54 |
StevenK | wgrant: Line 12 of yui/calendar-base/assets/skins/sam/calendar-base-skin.css | 05:55 |
wgrant | That's what I guessed from looking at the file. | 05:56 |
wgrant | StevenK: btw, if you want to purge yui 3.5.1, it's probably safe now | 05:57 |
wgrant | StevenK: http://mail.python.org/pipermail/python-announce-list/2009-August/007679.html | 05:58 |
wgrant | Search for DXImage | 05:59 |
wgrant | >>> parser.parseString(".yui3-skin-sam .yui3-calendar-content {\nfilter: progid:DXImageTransform.Microsoft.gradient( startColorstr='#f9f9f9', endColorstr='#f2f2f2',GradientType=0 ); /* IE6-9 */\n}").cssText | 06:00 |
StevenK | I see that | 06:00 |
wgrant | WARNINGProperty: Unknown Property name. [2:1: filter] | 06:00 |
wgrant | Seems to work with that enabled :) | 06:00 |
wgrant | '.yui3-skin-sam .yui3-calendar-content {\n filter: progid:DXImageTransform.Microsoft.gradient(startColorstr="#f9f9f9", endColorstr="#f2f2f2", GradientType=0);\n /* IE6-9 */\n }' | 06:00 |
StevenK | wgrant: I need the .set line? | 06:01 |
wgrant | StevenK: Yes, before you instantiate the CSSParser | 06:01 |
wgrant | Or you may be able to set it on the CSSParser itself | 06:01 |
wgrant | Not sure | 06:01 |
wgrant | (we are using the latest cssutils as of last week, btw, so an upgrade won't help) | 06:02 |
=== almaisan-away is now known as al-maisan | ||
StevenK | wgrant: http://pastebin.ubuntu.com/5591715/ | 06:06 |
wgrant | StevenK: Looks reasonable. Push and I'll rereview? | 06:11 |
StevenK | Oh, wait, I can go back to the original CSS | 06:17 |
StevenK | wgrant: Sorry, was on the phone since :08 | 06:18 |
StevenK | I'll check if the original CSS works | 06:18 |
wgrant | StevenK: good point | 06:22 |
StevenK | It does | 06:22 |
StevenK | wgrant: Changes pushed | 06:22 |
StevenK | wgrant: And diff updated | 06:25 |
wgrant | StevenK: + 'yui/calendar/assets/skins/sam/calendar.css', | 06:28 |
wgrant | Why aren't you using the one in yui/assets/skins/sam like the others? | 06:29 |
StevenK | wgrant: Good point | 06:30 |
StevenK | wgrant: Pushed that too | 06:32 |
StevenK | wgrant: Diff updated | 06:35 |
StevenK | wgrant: We no longer build 3.5.1, or do you want it dropped from sourcedeps? | 06:35 |
StevenK | % grep '^YUI_VER' Makefile | 06:36 |
StevenK | YUI_VERSIONS := 3.9.1 | 06:36 |
wgrant | StevenK: r=me | 06:36 |
wgrant | Oh, did you already do that? | 06:36 |
wgrant | I forgot | 06:36 |
wgrant | So you did | 06:36 |
wgrant | :) | 06:36 |
StevenK | r16573 | 06:37 |
StevenK | % bzr di | diffstat -s | 06:47 |
StevenK | 80 files changed, 1 insertion(+), 26238 deletions(-) | 06:47 |
StevenK | That felt good. | 06:47 |
wgrant | :) | 06:47 |
StevenK | wgrant: The checkwatches cronspam was python warnings? | 06:49 |
wgrant | StevenK: Yeah | 06:51 |
wgrant | There's also one Trac instance that returns truncated lines | 06:51 |
wgrant | But the main spam was from the RT deprecation warnings | 06:51 |
StevenK | Truncated lines is just awesome | 06:51 |
wgrant | That was the third most common avoidable cronspam on launchpad-error-reports, behind the process-upload encoding issues (which I fixed last week) and the process-upload build for superseded source exceptions (which are as-yet unfixed, but tolerable) | 06:52 |
StevenK | So it's still a firehose? | 06:52 |
wgrant | Much much better now that the process-upload encoding issue is fixed | 06:53 |
wgrant | StevenK: I wouldn't land the YUI2 removal until the new calendar is deployed and we're happy with it | 07:02 |
wgrant | Just in case we need to revert | 07:03 |
StevenK | Sure | 07:03 |
StevenK | But I'd thought I'd put it up | 07:03 |
StevenK | Mostly so I can gloat | 07:03 |
wgrant | Heh | 07:04 |
=== Gwaihir_ is now known as Gwaihir | ||
=== al-maisan is now known as almaisan-away | ||
=== wedgwood_away is now known as wedgwood | ||
=== almaisan-away is now known as al-maisan | ||
=== al-maisan is now known as almaisan-away | ||
=== deryck is now known as deryck[lunch] | ||
=== deryck[lunch] is now known as deryck |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!