Hey @erwinmombay, @jridgewell! These files were changed:

build-system/babel-plugins/babel-plugin-transform-jss/index.js

Hey @rsimha! These files were changed:

build-system/common/transform-cache.js

• Can we continue to assume 1 JSS file per component? Do we need to support multiple? If so, should we concat all their results into a single file?

Two components have multiple JSS files:

• InlineGallery separately styles the Pagination component because that may not always be used.
• amp-video has a separate autoplay.jss.js. Note also that VideoWrapper/VideoIframe are not being released on npm currently.

I'd recommend we concat into a single file for the sake of simplicity and given neither of the additional style files are particularly large even if we do end up publishing the videos.

As a naming nit, WDYT about styles.css or component.css as opposed to amp-*.css, given these are Preact components and not custom elements?

• Can we continue to assume 1 JSS file per component? Do we need to support multiple? If so, should we concat all their results into a single file?

Two components have multiple JSS files:

• InlineGallery separately styles the Pagination component because that may not always be used.
• amp-video has a separate autoplay.jss.js. Note also that VideoWrapper/VideoIframe are not being released on npm currently.

I'd recommend we concat into a single file for the sake of simplicity and given neither of the additional style files are particularly large even if we do end up publishing the videos.

As a naming nit, WDYT about styles.css or component.css as opposed to amp-*.css, given these are Preact components and not custom elements?

Thanks for the info. I've implemented concatting the JSS files, as well as renamed the file to styles.css