Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Richard Gibson
    @gibson042
    no, SGTM
    Timmy Willison
    @timmywil
    ok, thanks
    cu next time!
    we might not have many available this week
    which is okay. I have questions for mgol that can wait until next week.
    Dave Methvin
    @dmethvin
    Sorry I'm here but on a phone and gitter is flakey here
    Timmy Willison
    @timmywil
    it's okay. we'll postpone
    Michał Gołębiowski-Owczarek
    @mgol
    @timmywil I’m here today. :)
    heya
    Michał Gołębiowski-Owczarek
    @mgol
    Hi
    Timmy Willison
    @timmywil
    might just be you and me, but that's okay
    Richard Gibson
    @gibson042
    I'm here today
    Timmy Willison
    @timmywil
    the first thing is just a reminder for gibson to review jquery/sizzle#453
    Richard Gibson
    @gibson042
    nothing to report though; I haven't had time for that or anything jQuery for a while
    Timmy Willison
    @timmywil
    that's okay
    Richard Gibson
    @gibson042
    even to check what that change does with :has()
    Timmy Willison
    @timmywil
    For jquery/jquery#4463, gibson and I talked about this and agreed we should leave as-is
    Michał Gołębiowski-Owczarek
    @mgol
    @gibson042 That change has landed on jQuery master, though, does that mean your doubts apply there as well?
    Timmy Willison
    @timmywil
    wanted to get mgol's approval before closing
    Richard Gibson
    @gibson042
    "doubts" is a little strong, but yes
    Michał Gołębiowski-Owczarek
    @mgol
    I can report a Firefox issue
    Timmy Willison
    @timmywil
    cool
    jquery/jquery-mousewheel#202 I'm not really comfortable with the proposed workarounds here.
    Michał Gołębiowski-Owczarek
    @mgol
    I see. My (&, I assume, Dave's) idea was that it's not core but a plugin so we can allow a less ideal & temporary API here. But I understand the push against it as well.
    Timmy Willison
    @timmywil
    side note: I thought the mousewheel plugin would be EOL soon considering modern browser support for wheel
    Michał Gołębiowski-Owczarek
    @mgol
    especially that wheel events are one of the main reason for scroll jank
    hmm
    I think it has a bit more features than just normalizing events. But I'm not sure how important they are
    it'd be good to have Dave for this discussion here :)
    Timmy Willison
    @timmywil
    good point
    ok, we'll ask him next time we see him
    any reason we can't use offsetWidth/Height for documentElement?
    just to make it consistent that we don't use client
    Michał Gołębiowski-Owczarek
    @mgol
    offset* includes border, padding & scrollbars, client* only padding of those
    I think we're getting clientWidth specifically to include scrollbars, aren't we?
    Timmy Willison
    @timmywil
    yea
    so client must include those as well?
    Michał Gołębiowski-Owczarek
    @mgol
    sorry, to not include scrollbars
    so that you get full available document width for your layout purposes
    if I remember it correctly
    Timmy Willison
    @timmywil
    alright, well it sounds like we can close this issue without changes then
    Michał Gołębiowski-Owczarek
    @mgol
    yea
    Timmy Willison
    @timmywil
    +1
    so weird
    Michał Gołębiowski-Owczarek
    @mgol
    haha