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
Control element lifecycle with height auto
  • Loading branch information
@mszylkowski
mszylkowski committed Sep 23, 2021
commit e824e7f52e45d5f783820cafb2a174af4ab88511
4 changes: 4 additions & 0 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 @@ -117,6 +118,9 @@ describes.realWin(
});

AmpStory.isBrowserSupported = () => true;

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

afterEach(() => {
Expand Down