Paul Jolly

Results 610 comments of Paul Jolly

Here are complete repro steps: ``` git fetch https://github.com/myitcvforks/play-with-docker refs/heads/remove_container_name_hard_coding && git checkout FETCH_HEAD docker-compose down -v docker-compose up ``` Then: * Browse to http://localhost/ * Click "Start" At this...

@JamesMGreene - it does indeed... with one small extra change. If `obj.getBoundingClientRect` is defined, `info` should also take its height and width from it: ``` javascript if (typeof obj.getBoundingClientRect !==...

I spoke too soon. Seems like this will not work in Chrome until [this bug gets fixed](https://bugs.webkit.org/show_bug.cgi?id=105979)

@JamesMGreene - sorry, should have messaged you directly with my last comment. Just to make sure you saw it (regarding the bug in webkit)

You are right; my original post was lazy. I don't know the inner workings of [reveal.js](https://github.com/hakimel/reveal.js) but I believe it does rely on CSS3 `transform` as opposed to the ancient...

@JamesMGreene - sorry for the delay. Looks good in Firefox, but not Chrome. Will investigate and see if I can't work out the issue. Is there some way we can...

I think I've tracked down the problem (at least my problem when using ZeroClipboard with reveal.js). The key to the problem is [this line in reveal.js](https://github.com/hakimel/reveal.js/blob/f1234e51e1ec9fd299b6f2f5705d5a5b778a0840/js/reveal.js#L911) reveal.js uses zoom in...

@JamesMGreene re [your comment](https://github.com/zeroclipboard/ZeroClipboard/issues/149#issuecomment-22727455) on jQuery, I'm not surprised. Probably caused by the same [underlying bug](https://bugs.webkit.org/show_bug.cgi?id=105979)

Hmm, in which case I think I've linked to the wrong bug. Whichever of the bugs alluded to in that chain which causes `getBoundingClientRect` to be wrong in Chrome is...

@JamesMGreene - just a link comment, referencing [my comment](https://github.com/zeroclipboard/zeroclipboard-rails/pull/10#issuecomment-28147524) that refers to the Gist where I made CSS zoom work: https://gist.github.com/myitcv/7396290