Attending: timmywil, markelog, gibson042, mgol, jbedard

Location: https://gitter.im/jquery/meeting

• No meeting May 22 or May 29, Timmy on vacation, Memorial Day

link Issues and PRs needing review

• https://github.com/jquery/jquery/issues/3664 disabled on anchor/click event
  • Don’t put disabled on an anchor
• https://github.com/jquery/jquery/pull/3656 include scroll gutter in width/height
  • LGTM
• https://github.com/jquery/jquery/pull/3657 Deferred memory leak
  • LGTM - I’m not sure how to test this cleanly either
  • Gibson to take a look
• https://github.com/jquery/jquery/pull/3661 IE/Edge support comments
  • LGTM
• https://github.com/jquery/jquery/pull/3666 param + array
  • Seems odd to get this now
  • Probably can’t change this

link 3.3 issues

• https://github.com/jquery/jquery/issues?page=1&q=is%3Aopen+is%3Aissue+milestone%3A3.3.0
  • Needs assignment
• https://github.com/jquery/jquery/issues/3606
  • Check your assignments. Pick up someone else’s if you have time.

Attending: timmywil, mgol, jbedard, gibson042, markelog, dmethvin

Location: https://gitter.im/jquery/meeting

link Issues and PRs needing review

• https://github.com/jquery/jquery/issues/3642 math tag clone
  • Gibson042 to look into selector issue
• https://github.com/jquery/jquery/pull/3617 isFunction
  • Landed
• https://github.com/jquery/jquery/pull/3590 xhr: add ontimeout callback
  • LGTM
• https://github.com/jquery/jquery/pull/3569 CSS: drop float mapping
  • LGTM
• https://github.com/jquery/jquery/pull/3487 offset/position
  • Landed

link Edge testing - 14+ right now

link Consider dropping IE9/10 in jQuery 4.0.

• Coordinate with Angular, Ember, and other libs.

link 3.3 issues

• https://github.com/jquery/jquery/issues?page=1&q=is%3Aopen+is%3Aissue+milestone%3A3.3.0
• Needs assignment
  • https://github.com/jquery/jquery/issues/3606
• Check your assignments. Pick up someone else’s if you have time. Assigned count:
  • timmywil: 11
  • dmethvin: 7
  • gibson042: 8
  • mgol: 3
  • markelog: 2
  • jbedard: 0

Attending: timmywil, gibson042

Location: https://gitter.im/jquery/meeting

link Issues and PRs needing review

• https://github.com/jquery/jquery/pull/3617 isFunction
  • Looks like some small changes suggested from markelog to finish before merge. Anything outstanding to talk about?
• https://github.com/jquery/jquery/pull/3590 xhr: add ontimeout callback
  • LGTM
• https://github.com/jquery/jquery/pull/3569 CSS: drop float mapping
  • LGTM
• https://github.com/jquery/jquery/pull/3487 offset/position
  • Ready to land?

link 3.3 issues

• https://github.com/jquery/jquery/issues?page=1&q=is%3Aopen+is%3Aissue+milestone%3A3.3.0

• Needs assignment

  • https://github.com/jquery/jquery/issues/3606

• Check your assignments. Pick up someone else’s if you have time. Assigned count:

  • timmywil: 11
  • dmethvin: 7
  • gibson042: 8
  • mgol: 3
  • markelog: 2
  • jbedard: 0

Attending: timmywil, gibson042, markelog, dmethvin

Location: https://gitter.im/jquery/meeting

link Issues and PRs needing review

• https://github.com/jquery/jquery/issues/3623 licensing
  • Can we close this?
  • Pinged legal, waiting for answer
• https://github.com/jquery/jquery/pull/3617 isFunction
  • Markelog approve?
• https://github.com/jquery/jquery/issues/3589 innerWidth + scrollbars
  • What’s the resolution here? Milestone?
  • Try gibson’s solution of using offsetWidth to calculate scrollbar width and include with padding where applicable

link 3.3 issues

• https://github.com/jquery/jquery/issues?page=1&q=is%3Aopen+is%3Aissue+milestone%3A3.3.0
• Needs assignment
  • https://github.com/jquery/jquery/issues/3606
  • https://github.com/jquery/jquery/issues/3603
• Check your assignments. Pick up someone else’s if you have time. Assigned count:
  • timmywil: 11
  • dmethvin: 7
  • gibson042: 8
  • mgol: 3
  • markelog: 2
  • jbedard: 0

Attending: timmywil, markelog, gibson042, mgol

Location: https://gitter.im/jquery/meeting

link General workflow

• PRs can address multiple things, but commits should be atomic when landing
• When adding new tests, consider separating them from old ones based on convenience and whether it is a regression.

link Issues and PRs needing review

• https://github.com/jquery/jquery/pull/3617 isFunction
  • Support generators and async
  • Definitely necessary for Deferred even if isFunction is deprecated
• https://github.com/jquery/jquery/issues/3589 innerWidth + scrollbars
  • What’s the resolution here? Milestone?

link 3.3 issues

• https://github.com/jquery/jquery/issues?page=1&q=is%3Aopen+is%3Aissue+milestone%3A3.3.0
• Needs assignment
  • https://github.com/jquery/jquery/issues/3602
  • https://github.com/jquery/jquery/issues/3606
  • https://github.com/jquery/jquery/issues/3603
• Check your assignments. Pick up someone else’s if you have time. Assigned count:
  • timmywil: 10
  • dmethvin: 7
  • gibson042: 7
  • mgol: 3
  • markelog: 2
  • jbedard: 0

Attending: timmywil, dmethvin, mgol, jbedard, markelog, gibson042

Location: https://gitter.im/jquery/meeting

link Issues needing review

• https://github.com/jquery/jquery/issues/3600 isFunction on object tag
  • +1 to deprecating, audit internal usage
  • We might care about generators/async/proxy in some places, but not all
• https://github.com/jquery/jquery/issues/3603 Chromium memory bug
  • Wait for Chromium team’s response?
• https://github.com/jquery/jquery/issues/3606
  • Should be able to release handlers, right?

link Our approach to WordPress?

• Dave to reach out again about their upgrade plan

link 3.3 issues

• https://github.com/jquery/jquery/issues?page=1&q=is%3Aopen+is%3Aissue+milestone%3A3.3.0
• Needs assignment
  • https://github.com/jquery/jquery/issues/3605
  • https://github.com/jquery/jquery/issues/3609
• All issues have assignments! Check your assignments. Pick up someone else’s if you have time. Assigned count:
  • timmywil: 10
  • dmethvin: 7
  • gibson042: 7
  • mgol: 3
  • markelog: 2
  • jbedard: 0

Attending: timmywil, markelog, gibson042, jbedard, mgol

Location: https://gitter.im/jquery/meeting

link 3.2.1 Released!

• Thanks for your help in getting this out quickly.

link Issues needing review

• https://github.com/jquery/jquery/issues/3589 innerWidth/Height + scrollbars
  • If it’s never been consistent (due to IE), what do we do? Anything?
  • Not sure. Check computed css spec. Why do browsers differ?
• https://github.com/jquery/jquery/issues/3558 mouseleave + select
  • We already have a contains check. If related is null, what do we do?
  • Closed wontfix
• https://github.com/jquery/jquery/issues/3553 Array Iterator type + Babel
  • Map this type to array for Babel’s sake?
  • Needs test case or bug link

link PRs

• https://github.com/jquery/jquery/pulls?q=is%3Apr+is%3Aopen
• https://github.com/jquery/jquery/pull/3590 ontimeout + safari
  • LGTM
• https://github.com/jquery/jquery/pull/3569 CSS: drop float mapping
  • LGTM

link 3.3 issues

• https://github.com/jquery/jquery/issues?page=1&q=is%3Aopen+is%3Aissue+milestone%3A3.3.0
• All issues have assignments! Check your assignments. Pick up someone else’s if you have time. Assigned count:
  • timmywil: 9
  • dmethvin: 7
  • gibson042: 7
  • mgol: 2
  • markelog and jbedard: 0

Attending: timmywil, mgol, gibson042, dmethvin, jbedard

Location: https://gitter.im/jquery/meeting

link 3.2 Released! ...with some bugs

• https://github.com/jquery/jquery/issues?q=is%3Aopen+is%3Aissue+milestone%3A3.2.1
• Fixes for 2 regressions already landed
• Release today!
• https://github.com/jquery/jquery/issues/3579 a new one in trigger
  • Revert for now, fix in 3.3 by updating https://github.com/jquery/jquery/pull/1367
  • See https://github.com/jquery/jquery/pull/3581

Attending: timmywil, mgol, markelog, jbedard, gibson042

Location: https://gitter.im/jquery/meeting

link 3.2 Remaining Issues

• https://github.com/jquery/jquery/issues?q=is%3Aopen+is%3Aissue+milestone%3A3.2.0
• https://github.com/jquery/jquery/issues/3503 Animations get skipped
  • Push to 3.3? depending on whether gibson gets to it
• https://github.com/jquery/jquery/issues/3193 width & height + transforms
  • Timmy to finish today

link PRs

• https://github.com/jquery/jquery/pulls?q=is%3Aopen+is%3Apr+milestone%3A3.2.0
• https://github.com/jquery/jquery/pull/3487 offset & position
  • Need to either land or push to 3.3
• https://github.com/jquery/jquery/pull/3151 raf timestamps
  • Can we land this?
• https://github.com/jquery/jquery/pull/3557 CSS vars
  • LGTM
  • Kebabcase change pending