Skip to content

Commit

Permalink
Merge pull request #583 from PolymerElements/no-invalid-typedef
Browse files Browse the repository at this point in the history
This code was using @typedef like @type, which is invalid
  • Loading branch information
rictic authored Jan 21, 2020
2 parents 275c510 + 4e38e15 commit 5882cd2
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 24 deletions.
5 changes: 1 addition & 4 deletions app-scroll-effects/effects/blend-background.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,7 @@ registerEffect('blend-background', {
class BlendBackground {
constructor() {
/**
* @typedef {{
* backgroundFrontLayer: !HTMLElement,
* backgroundRearLayer: !HTMLElement,
* }}
* @type {?}
*/
this._fxBlendBackground;
}
Expand Down
5 changes: 1 addition & 4 deletions app-scroll-effects/effects/fade-background.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,10 +57,7 @@ registerEffect('fade-background', {
class FadeBackground {
constructor() {
/**
* @typedef {{
* backgroundFrontLayer: !HTMLElement,
* backgroundRearLayer: !HTMLElement,
* }}
* @type {?}
*/
this._fxFadeBackground;
}
Expand Down
6 changes: 1 addition & 5 deletions app-scroll-effects/effects/parallax-background.js
Original file line number Diff line number Diff line change
Expand Up @@ -69,11 +69,7 @@ class ParallaxBackground {
this._dHeight;

/**
* @typedef {{
* background: !HTMLElement,
* backgroundFrontLayer: !HTMLElement,
* backgroundRearLayer: !HTMLElement,
* }}
* @type {?}
*/
this._fxParallaxBackground;
}
Expand Down
5 changes: 1 addition & 4 deletions app-scroll-effects/effects/resize-snapped-title.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,10 +70,7 @@ registerEffect('resize-snapped-title', {
class ResizeSnappedTitle {
constructor() {
/**
* @typedef {{
* title: !HTMLElement,
* condensedTitle: !HTMLElement,
* }}
* @type {?}
*/
this._fxResizeSnappedTitle;
}
Expand Down
8 changes: 1 addition & 7 deletions app-scroll-effects/effects/resize-title.js
Original file line number Diff line number Diff line change
Expand Up @@ -103,13 +103,7 @@ class ResizeTitle {
this.condenses;

/**
* @typedef {{
* title: !HTMLElement,
* condensedTitle: !HTMLElement,
* scale: number,
* titleDX: number,
* titleDY: number,
* }}
* @@type {?}
*/
this._fxResizeTitle;
}
Expand Down

0 comments on commit 5882cd2

Please sign in to comment.