Skip to content

DOM update February 2019 #29690

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Feb 4, 2019
Merged

DOM update February 2019 #29690

merged 4 commits into from
Feb 4, 2019

Conversation

sandersn
Copy link
Member

@sandersn sandersn commented Feb 1, 2019

Updates from TSJS-lib-generator. Notably, this update includes JSDoc from MDN.

@sandersn
Copy link
Member Author

sandersn commented Feb 4, 2019

Ooops, I forgot to update baselines. I just did, for all the correct changes.

Unfortunately, the test for over-large inferred reverse-mapped types now hits the inference cut off instead of succeeding as it did after the fix that it was added with. I need to decide if this is correct or not.

@sandersn sandersn merged commit 2f9218f into master Feb 4, 2019
@sandersn sandersn deleted the dom-update-feb-2019 branch February 4, 2019 23:25
@@ -2176,6 +2214,7 @@ declare var BhxBrowser: {
new(): BhxBrowser;
};

/** The BiquadFilterNode interface represents a simple low-order filter, and is created using the AudioContext.createBiquadFilter() method. It is an AudioNode that can represent different kinds of filters, tone control devices, and graphic equalizers. */
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't it improve readability+quality to cut leading The BiquadFilterNode interface?

@@ -2145,6 +2182,7 @@ declare var BaseAudioContext: {
new(): BaseAudioContext;
};

/** The beforeunload event is fired when the window, the document and its resources are about to be unloaded. */
Copy link
Contributor

Choose a reason for hiding this comment

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

Or here drop The beforeunload event is

@@ -2190,6 +2229,7 @@ declare var BiquadFilterNode: {
new(context: BaseAudioContext, options?: BiquadFilterOptions): BiquadFilterNode;
};

/** A Blob object represents a file-like object of immutable, raw data. Blobs represent data that isn't necessarily in a JavaScript-native format. The File interface is based on Blob, inheriting blob functionality and expanding it to support files on the user's system. */
Copy link
Contributor

Choose a reason for hiding this comment

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

Here drop A Blob object

interface CSS {
escape(value: string): string;
supports(property: string, value?: string): boolean;
}
declare var CSS: CSS;

/** An object implementing the CSSConditionRule interface represents a single condition CSS at-rule, which consists of a condition and a statement block. It is a child of CSSGroupingRule. */
Copy link
Contributor

Choose a reason for hiding this comment

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

Here a bit more complex: An object implementing the CSSConditionRule interface

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants