Skip to content

🚀 [Story embeds] Remove support for expanded components like amp-twitter #36851

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 14 commits into from
Nov 11, 2021
Merged

🚀 [Story embeds] Remove support for expanded components like amp-twitter #36851

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
e4f1956
Working removal
mszylkowski Nov 8, 2021
b5db57f
Removed extra styles / vars
mszylkowski Nov 8, 2021
fc37c15
Keep removing
mszylkowski Nov 9, 2021
e58de03
Removed more
mszylkowski Nov 9, 2021
94898c0
Merge branch 'main' of github.com:ampproject/amphtml into remove_embe…
mszylkowski Nov 9, 2021
44d80f2
Started removing languages
mszylkowski Nov 9, 2021
001344c
Removing localizatino strings
mszylkowski Nov 9, 2021
5c8ebf0
Removed other localizations
mszylkowski Nov 9, 2021
a2541aa
Removed remaining expand tweet
mszylkowski Nov 9, 2021
84d53e5
Updated zindex
mszylkowski Nov 9, 2021
29fdca8
Removed visual tests
mszylkowski Nov 10, 2021
0c9113a
Removed unused const
mszylkowski Nov 10, 2021
cd10a57
Adding click shield to tweet
mszylkowski Nov 10, 2021
f2aa242
Merge branch 'main' of github.com:ampproject/amphtml into remove_embe…
mszylkowski Nov 10, 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
Adding click shield to tweet
  • Loading branch information
@mszylkowski
mszylkowski committed Nov 10, 2021
commit cd10a57027e68966afb07a24f9a4f8cc4bc80e6f
4 changes: 1 addition & 3 deletions extensions/amp-story/1.0/amp-story-embedded-component.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,7 @@ function getComponentSelectors(components) {
*/
export function embeddedElementsSelectors() {
// Using indirect invocation to prevent no-export-side-effect issue.
return Object.values(
getComponentSelectors(EMBEDDED_COMPONENTS_SELECTORS)
).join(',');
return Object.keys(EMBEDDED_COMPONENTS_SELECTORS).join(',');
}

/**
Expand Down