Skip to content
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.

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

[SyntheticModules] Implements CSS Modules #20012

Merged
merged 1 commit into from Dec 14, 2019

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Oct 31, 2019

This is the final change required for CSS Modules to be utilized by developers.
Following the acceptance of this change, if you run chromium with the CSSModules runtime flag, the following is now valid syntax:

<script type="module">
import sheet from "./example.css";
</script>

CSS Modules Explainer:
https://github.com/w3c/webcomponents/blob/gh-pages/proposals/css-modules-v1-explainer.md

CSS Modules Spec PR:
whatwg/html#4898

Bug: 967018
Change-Id: Ifdee5b92259fb7e4e9c8f9aa88e69a98eb55c551
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799923
Commit-Queue: Sam Sebree <sasebree@microsoft.com>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#724896}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1799923 branch 5 times, most recently from b1ea461 to 2c34130 Compare November 6, 2019 19:02
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title [SyntheticModules] Implements CSS Modules address feedback Nov 9, 2019
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title address feedback [SyntheticModules] Implements CSS Modules Nov 11, 2019
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1799923 branch 4 times, most recently from 2ce9067 to 115a9a2 Compare November 21, 2019 23:40
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1799923 branch 3 times, most recently from 9955e88 to c193418 Compare November 26, 2019 06:03
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1799923 branch 2 times, most recently from b8ca5e7 to 3236b59 Compare December 10, 2019 05:37
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1799923 branch 5 times, most recently from 8694fc6 to 408d102 Compare December 14, 2019 00:55
This is the final change required for CSS Modules to be utilized by developers.
Following the acceptance of this change, if you run chromium with the CSSModules runtime flag, the following is now valid syntax:

<script type="module">
    import sheet from "./example.css";
</script>

CSS Modules Explainer:
https://github.com/w3c/webcomponents/blob/gh-pages/proposals/css-modules-v1-explainer.md

CSS Modules Spec PR:
whatwg/html#4898

Bug: 967018
Change-Id: Ifdee5b92259fb7e4e9c8f9aa88e69a98eb55c551
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799923
Commit-Queue: Sam Sebree <sasebree@microsoft.com>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#724896}
@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 6fbd872 into master Dec 14, 2019
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-1799923 branch December 14, 2019 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants