Skip to content

🖍 [Story performance] Remove standalone classes #36127

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.

Sign up for GitHub

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
mszylkowski merged 18 commits into from
Oct 8, 2021
Merged

🖍 [Story performance] Remove standalone classes #36127

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
278b389
Remove standalone classes
mszylkowski Sep 21, 2021
8b16778
Updated specificity
mszylkowski Sep 22, 2021
43840c2
Added specificity again
mszylkowski Sep 22, 2021
9eb6688
Merge branch 'main' of github.com:ampproject/amphtml into remove_runt…
mszylkowski Sep 22, 2021
b7b7036
Merge branch 'main' of github.com:ampproject/amphtml into remove_runt…
mszylkowski Sep 22, 2021
41575d6
Remove standalone classes
mszylkowski Sep 21, 2021
c519391
Updated specificity
mszylkowski Sep 22, 2021
cd44147
Added specificity again
mszylkowski Sep 22, 2021
2b47f13
Merge branch 'remove_runtimeclasses' of github.com:mszylkowski/amphtm…
mszylkowski Sep 22, 2021
e824e7f
Control element lifecycle with height auto
mszylkowski Sep 23, 2021
4d8805c
Started changing tests to use AmpStory
mszylkowski Sep 24, 2021
2636c3a
Reverted to automatic height
mszylkowski Oct 5, 2021
e87efc4
Remove unused test setup
mszylkowski Oct 5, 2021
c39680f
Merge branch 'main' of github.com:ampproject/amphtml into remove_runt…
mszylkowski Oct 5, 2021
37f0a80
Updated comment to make it more clear
mszylkowski Oct 6, 2021
cade19e
Merge branch 'main' of github.com:ampproject/amphtml into remove_runt…
mszylkowski Oct 7, 2021
50476ef
Remove unused rule
mszylkowski Oct 7, 2021
504c12f
Merge branch 'main' of github.com:ampproject/amphtml into remove_runt…
mszylkowski Oct 8, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Reverted to automatic height
  • Loading branch information
@mszylkowski
mszylkowski committed Oct 5, 2021
commit 2636c3a500e7038e094c81e9721758eae7c07d2c
19 changes: 6 additions & 13 deletions extensions/amp-story/1.0/test/test-amp-story.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import {createElementWithAttributes} from '#core/dom';
import {registerServiceBuilder} from '../../../../src/service-helpers';
import {toggleExperiment} from '#experiments';
import {waitFor} from '#testing/test-helper';
import {setImportantStyles} from '#core/dom/style';

// Represents the correct value of KeyboardEvent.which for the Right Arrow
const KEYBOARD_EVENT_WHICH_RIGHT_ARROW = 39;
Expand Down Expand Up @@ -51,7 +52,7 @@ describes.realWin(
* @param {Array<string>=} ids
* @return {!Array<!Element>}
*/
function createStoryWithPages(count, ids = [], autoAdvance = false) {
async function createStoryWithPages(count, ids = [], autoAdvance = false) {
element = win.document.createElement('amp-story');
element.getAmpDoc = () => env.ampdoc;

Expand All @@ -68,7 +69,7 @@ describes.realWin(
});

win.document.body.appendChild(element);
story = new AmpStory(element);
story = await element.getImpl();

return pageArray;
}
Expand Down Expand Up @@ -118,6 +119,9 @@ describes.realWin(
});

AmpStory.isBrowserSupported = () => true;

/** Control element lifecycle by preventing automatic callback calls. */
setImportantStyles(win.document.documentElement, {'height': 'auto'});
});

afterEach(() => {
Expand All @@ -127,15 +131,12 @@ describes.realWin(
it('should build with the expected number of pages', async () => {
const pagesCount = 2;
await createStoryWithPages(pagesCount, ['cover', 'page-1']);
await story.buildCallback();
await story.layoutCallback();
expect(story.getPageCount()).to.equal(pagesCount);
});

it('should activate the first page when built', async () => {
await createStoryWithPages(2, ['cover', 'page-1']);

await story.buildCallback();
await story.layoutCallback();
// Getting all the AmpStoryPage objets.
const pageElements = story.element.getElementsByTagName('amp-story-page');
Expand Down Expand Up @@ -165,8 +166,6 @@ describes.realWin(
await createStoryWithPages(2);

const buildShareMenuStub = env.sandbox.stub(story.shareMenu_, 'build');

await story.buildCallback();
await story.layoutCallback();
expect(buildShareMenuStub).to.have.been.calledOnce;
});
Expand All @@ -189,8 +188,6 @@ describes.realWin(

it('should pause/resume pages when switching pages', async () => {
await createStoryWithPages(2, ['cover', 'page-1']);

await story.buildCallback();
await story.layoutCallback();
// Getting all the AmpStoryPage objects.
const pageElements = story.element.getElementsByTagName('amp-story-page');
Expand Down Expand Up @@ -242,8 +239,6 @@ describes.realWin(

it('lock body when amp-story is initialized', async () => {
await createStoryWithPages(2, ['cover', 'page-1']);

await story.buildCallback();
await story.layoutCallback();
story.lockBody_();
expect(win.document.body.style.getPropertyValue('overflow')).to.be.equal(
Expand All @@ -256,8 +251,6 @@ describes.realWin(

it('checks if pagination buttons exist ', async () => {
await createStoryWithPages(2, ['cover', 'page-1']);

await story.buildCallback();
await story.layoutCallback();
expect(
story.element.querySelectorAll('.i-amphtml-story-button-container')
Expand Down