Skip to content

Add compat data for ImageBitmapRenderingContext#885

Merged
Elchi3 merged 2 commits into
mdn:masterfrom
maboa:ImageBitmapRenderingContext
Jan 25, 2018
Merged

Add compat data for ImageBitmapRenderingContext#885
Elchi3 merged 2 commits into
mdn:masterfrom
maboa:ImageBitmapRenderingContext

Conversation

@maboa

@maboa maboa commented Jan 22, 2018

Copy link
Copy Markdown
Contributor

Wondering if there is a better way to represent :

transferImageBitmap() -> transferFromImageBitmap()

Also as there was no Safari 55.0 - I assumed that data point was incorrect (https://developer.mozilla.org/en-US/docs/Web/API/ImageBitmapRenderingContext)

@maboa maboa changed the title add compat data for ImageBitmapRenderingContext Add compat data for ImageBitmapRenderingContext Jan 22, 2018
@Elchi3 Elchi3 added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Jan 22, 2018

@Elchi3 Elchi3 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a typical case for the use of alternative_name.

Comment thread api/ImageBitmapRenderingContext.json Outdated
"deprecated": false
}
},
"transferImageBitmap() -> transferFromImageBitmap()": {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use "transferFromImageBitmap": { as an identifier here.
Then, for Firefox, please express the alternative name, that used to exist, like this:

"firefox": [
  {
   "version_added": "52"
  },
  {
    "version_added": "46",
    "version_removed": "52",
    "alternative_name": "transferImageBitmap"
  }
],

Same for firefox_android.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK - will do. Thanks

@maboa

maboa commented Jan 25, 2018

Copy link
Copy Markdown
Contributor Author

OK done :) I also added mdn_url and status to transferFromImageBitmap.

@Elchi3 Elchi3 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants