#172: ViewportAPI naming Re-review

Visit on Github.

Opened Apr 26, 2017

Hello TAG!

I'm requesting a TAG (re)review of:

You should also know that...

This was reviewed back in #128. The feedback mainly centred around naming of the API. We've updated the names to hopefully be less of a hodgepodge. I'd like to hear from TAG if this is better or if there are any concerns about the new names? No need to rereview anything else (but feedback is always appreciated).

Changes:

  • window.visualViewport --> window.view
  • scrollLeft, scrollTop --> offsetX, offsetY
  • clientWidth, clientHeight --> width, height

We'd prefer the TAG provide feedback as (please select one):

  • leave review feedback as a comment in this issue and @-notify [github usernames]

Discussions