Skip to content

Commit 9813cea

Browse files
authored
♻️ #core form, modal, amp-element-helpers ("real" diff in desc) (ampproject#35608)
* Update src/form code * Update src/modal code * Update src/video-interface code * Move src/{form,modal,amp-element-helpers} -> #core/dom * Fix types * update dep-check config * Remove bad assert * Update imports of form,modal,amp-element-helpers * Fix rebase error
1 parent 4ed4b92 commit 9813cea

50 files changed

Lines changed: 113 additions & 129 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

ads/google/a4a/utils.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ import {getPageLayoutBoxBlocking} from '#core/dom/layout/page-layout-box';
3030
import {getTimingDataSync} from '#service/variable-source';
3131
import * as mode from '#core/mode';
3232
import {parseJson} from '#core/types/object/json';
33-
import {whenUpgradedToCustomElement} from '../../../src/amp-element-helpers';
33+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
3434
import {createElementWithAttributes} from '#core/dom';
3535

3636
/** @type {string} */

build-system/test-configs/dep-check-config.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,6 @@ exports.rules = [
124124
// somewhere else at some point
125125
'ads/google/a4a/**->src/ad-cid.js',
126126
'ads/google/a4a/**->src/consent.js',
127-
'ads/google/a4a/**->src/amp-element-helpers.js',
128127
'ads/google/a4a/**->src/experiments/index.js',
129128
'ads/google/a4a/**->src/service/index.js',
130129
'ads/google/a4a/utils.js->src/service/variable-source.js',

extensions/amp-accordion/0.1/test/test-amp-accordion.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import {computedStyle} from '#core/dom/style';
2323
import {createElementWithAttributes, tryFocus} from '#core/dom';
2424
import {htmlFor} from '#core/dom/static-template';
2525
import {poll} from '#testing/iframe';
26-
import {whenUpgradedToCustomElement} from '../../../../src/amp-element-helpers';
26+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
2727

2828
describes.realWin(
2929
'amp-accordion',

extensions/amp-analytics/0.1/events.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ import {deepMerge, dict, hasOwn} from '#core/types/object';
2626
import {dev, devAssert, user, userAssert} from '../../../src/log';
2727
import {getData} from '../../../src/event-helper';
2828
import {getDataParamsFromAttributes} from '#core/dom';
29-
import {isAmpElement} from '../../../src/amp-element-helpers';
29+
import {isAmpElement} from '#core/dom/amp-element-helpers';
3030
import {isArray, isEnumValue, isFiniteNumber} from '#core/types';
3131
import {debounce} from '#core/types/function';
3232
import {isExperimentOn} from '#experiments';

extensions/amp-auto-ads/0.1/placement.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ import {createElementWithAttributes} from '#core/dom';
2828
import {dev, user} from '../../../src/log';
2929
import {dict} from '#core/types/object';
3030
import {measurePageLayoutBox} from './measure-page-layout-box';
31-
import {whenUpgradedToCustomElement} from '../../../src/amp-element-helpers';
31+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
3232

3333
/** @const */
3434
const TAG = 'amp-auto-ads';

extensions/amp-auto-lightbox/0.1/amp-auto-lightbox.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ import {loadPromise} from '../../../src/event-helper';
3333
import {measureIntersectionNoRoot} from '#core/dom/layout/intersection-no-root';
3434
import {toArray} from '#core/types/array';
3535
import {tryParseJson} from '#core/types/object/json';
36-
import {whenUpgradedToCustomElement} from '../../../src/amp-element-helpers';
36+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
3737

3838
const TAG = 'amp-auto-lightbox';
3939

extensions/amp-bind/0.1/bind-impl.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ import {escapeCssSelectorIdent} from '#core/dom/css-selectors';
3232
import {findIndex, isArray, remove, toArray} from '#core/types/array';
3333
import {getMode} from '../../../src/mode';
3434
import {iterateCursor} from '#core/dom';
35-
import {whenUpgradedToCustomElement} from '../../../src/amp-element-helpers';
35+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
3636

3737
import {invokeWebWorker} from '../../../src/web-worker/amp-worker';
3838
import {isAmp4Email} from '../../../src/format';

extensions/amp-brightcove/0.1/test/test-amp-brightcove.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ import {createElementWithAttributes} from '#core/dom';
2424
import {listenOncePromise} from '../../../../src/event-helper';
2525
import {macroTask} from '#testing/yield';
2626
import {parseUrlDeprecated} from '../../../../src/url';
27-
import {whenUpgradedToCustomElement} from '../../../../src/amp-element-helpers';
27+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
2828

2929
describes.realWin(
3030
'amp-brightcove',

extensions/amp-carousel/0.1/test/test-scrollable-carousel.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import {ActionTrust} from '#core/constants/action-constants';
2020
import {Services} from '#service';
2121
import {createElementWithAttributes} from '#core/dom';
2222
import {user} from '../../../../src/log';
23-
import {whenUpgradedToCustomElement} from '../../../../src/amp-element-helpers';
23+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
2424
describes.realWin(
2525
'test-scrollable-carousel',
2626
{

extensions/amp-carousel/0.1/test/test-slidescroll.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import {Services} from '#service';
2121
import {createElementWithAttributes} from '#core/dom';
2222
import {installResizeObserverStub} from '#testing/resize-observer-stub';
2323
import {user} from '../../../../src/log';
24-
import {whenUpgradedToCustomElement} from '../../../../src/amp-element-helpers';
24+
import {whenUpgradedToCustomElement} from '#core/dom/amp-element-helpers';
2525

2626
describes.realWin(
2727
'SlideScroll',

0 commit comments

Comments
 (0)