PEP Team Meeting – Sep 22 2016

jQuery UI Team Meeting – Sep 21 2016

Ticket triage
- 430 tickets total, +2
- 14 pull requests, +1
Sortable
- regression: cancel() method throws error
  - https://bugs.jqueryui.com/ticket/15042
  - Scott sent a PR
    - https://github.com/jquery/jquery-ui/pull/1746
    - merged
1.12.1
- Released!
Selectable
- classes option causing performance issues
  - https://bugs.jqueryui.com/ticket/15049
Autocomplete
- XSS vulnerability in remote JSONP demo
  - https://bugs.jqueryui.com/ticket/15048
  - Doesn't seem like a real vulnerability
  - PR submitted
    - https://github.com/jquery/jquery-ui/pull/1747
    - PR implementation is too restrictive, rejecting many valid function names
Datepicker
- Strange iOS 10 behavior
- https://bugs.jqueryui.com/ticket/15050
Download builder
- Intermittent issues

Attending: timmywil, DaveMethvin, jbedard, mgol, gibson042 Location: https://gitter.im/jquery/meeting

Ticket triage
- 428 tickets total, -0
- 13 pull requests, -0
Slider
- Add demo for custom handle
  - https://bugs.jqueryui.com/ticket/15023
  - Scott sent a PR
    - https://github.com/jquery/jquery-ui/pull/1740
    - merged
Icon Color
- Icons are getting the hover color by default
  - https://github.com/jquery/jqueryui.com/issues/159
  - Likely caused by this change: https://github.com/jquery/jquery-ui/commit/6e2bbcaa5e19bb933d8024ff5d6fcff34f91e034#diff-ff24a972bba1a2ecf79314fd9a5d048fR208
  - Alex sent a PR
    - https://github.com/jquery/jquery-ui/pull/1745
    - merged
Sortable
- regression: cancel() method throws error
  - https://bugs.jqueryui.com/ticket/15042
  - Scott sent a PR
    - https://github.com/jquery/jquery-ui/pull/1746
1.12.1
- Slider PR merged
- Removed externals from CDN zip
- Deployed updated API docs
- Fixed icon color
- Sortable cancel() PR needs to be merged
- Ready to release?
  - Yes

Attending: timmywil, markelog, dmethvin, mgol, gibson042 Location: https://gitter.im/jquery/meeting

https://github.com/jquery/jquery/issues?q=is%3Aopen+is%3Aissue+milestone%3A3.1.1
https://github.com/jquery/jquery/issues/3260 CDN .map
- Dave to work on it this week
https://github.com/jquery/jquery/issues/3072 & https://github.com/jquery/jquery/issues/3003
- In progress, PR incoming
https://github.com/jquery/jquery/issues/3204 Edge 14 whitespace error
- PR: https://github.com/jquery/jquery/pull/3308 needs review
- Close. The issue has caveats that we don’t think merit elaborate workarounds in Core.
- Dave to bug the Edge team to fix this.
- Either ignore the failing test for now or skip in code until it’s fixed

Need clarification on spec wording of "the element with the default touch behavior" for touch-action.
- For the common case of scrolling it's pretty simple - just walk up until you hit a scroller.
- If you add an overflow:scroll element as a descendant of a touch-action:none region, you should get scrolling back (consider component composition scenarios).
- As you walk up the ancestor chain, you're intersecting all the touch-actions.
- Chrome implementation for touch-action detection: https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/input/TouchActionUtil.cpp?q=computeEffectiveTouchAction&sq=package:chromium&l=47&dr=CSs
It’s not worth adding the passive event listeners because Safari will end up being the only browser without native PointerEvents and it doesn’t have native support for all touch-action values.
There’s nothing holding up the 0.4.2 release at this point.

Ticket triage
- 428 tickets total, -9
- 13 pull requests, -6
Spinner
- Not going to support Globalize 1.x for now
- Scott sent a PR to document the version support
- https://github.com/jquery/api.jqueryui.com/pull/307
- merged
API Site
- Alex updated the main site to show the 1.12 docs
Position
- Fractional value detection is failing in Chrome
- Scott sent a PR to remove the detection
- https://github.com/jquery/jquery-ui/pull/1739
- merged
Slider
- Add demo for custom handle
- https://bugs.jqueryui.com/ticket/15023
- Scott sent a PR
- https://github.com/jquery/jquery-ui/pull/1740
- Jörn noticed a bug in slider when reviewing the demo
- Scott fixed the bug as part of the PR
- Need another round of review
Icon Color
- Icons are getting the hover color by default
- https://github.com/jquery/jqueryui.com/issues/159
- Likely caused by this change:
- https://github.com/jquery/jquery-ui/commit/6e2bbcaa5e19bb933d8024ff5d6fcff34f91e034#diff-ff24a972bba1a2ecf79314fd9a5d048fR208
- Alex to look into why this change was made
1.12.1
- Slider PR
- Remove externals from CDN zip
- Deploy updated API docs
- Fix icon color

Marius created a PR for passive event listeners and touch-action
- https://github.com/jquery/PEP/pull/309
- Need clarification on "nearest ancestor with default touch behavior" from https://www.w3.org/TR/pointerevents/#h3_the-touch-action-css-property
Release a new version as soon as passive event listeners land

Ticket triage
- 437 tickets total, +0
- 19 pull requests, +0
Spinner
- Doesn't work with Globalize 1.x
  - https://bugs.jqueryui.com/ticket/15030
  - Will be tricky to support both because of Globalize.format() being split into multiple methods
Menu
- Doesn't work with accented characters
  - https://bugs.jqueryui.com/ticket/15038
  - This is exactly the problem we discussed with the numpad problem :-(
Position
- Fractional value detection is failing in Chrome
  - Just rip it out since it only applies to IE <=9
  - Scott to send a PR