Skip to content

Topics/regroup non static files #6346

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
m-allanson merged 18 commits into from
Jul 17, 2018
Merged

Topics/regroup non static files #6346

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
4803d61
First try at cleaning up build output
brotzky Jul 4, 2018
e1b8be9
Adding flag to build, ordering pathPrefix correctly, and cleaned up s…
brotzky Jul 4, 2018
77371b9
Removing the deletion of files as referenced by m-allanson and pieh
brotzky Jul 4, 2018
59cf18a
Adding support for styles grouping
brotzky Jul 4, 2018
366bd97
renaming output directories to js and css
brotzky Jul 6, 2018
972fb6b
First try at cleaning up build output
brotzky Jul 4, 2018
4042ae1
Adding flag to build, ordering pathPrefix correctly, and cleaned up s…
brotzky Jul 4, 2018
695cb68
Removing the deletion of files as referenced by m-allanson and pieh
brotzky Jul 4, 2018
699e5ca
Adding support for styles grouping
brotzky Jul 4, 2018
eed6616
renaming output directories to js and css
brotzky Jul 6, 2018
d8a8377
Merge branch 'topics/regroup-non-static-files' of github.com:narative…
brotzky Jul 6, 2018
87f0ada
Removing config flag
brotzky Jul 8, 2018
cd834ab
Removing config flag logic from styles
brotzky Jul 8, 2018
414ce25
Merge branch 'master' into topics/regroup-non-static-files
brotzky Jul 12, 2018
41f80b6
Removing 'text/css' from CSS Link
brotzky Jul 12, 2018
ef334d2
Merge branch 'topics/regroup-non-static-files' of github.com:narative…
brotzky Jul 12, 2018
6f1e987
Woops, adding back the CSS as I was testing locally and commited the …
brotzky Jul 12, 2018
f1ad758
Adding '../' to the relative static route similar to CSS
brotzky Jul 13, 2018
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
1 change: 0 additions & 1 deletion packages/gatsby/cache-dir/static-entry.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,6 @@ export default (pagePath, callback) => {
} else {
headComponents.unshift(
<style
Copy link
Contributor Author

@brotzky brotzky Jul 12, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

#6346 (comment) @m-allanson removed.

Copy link
Contributor

@m-allanson m-allanson Jul 12, 2018

Choose a reason for hiding this comment

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

Thanks 👍

type="text/css"
data-href={urlJoin(pathPrefix, style.name.slice(2))}
dangerouslySetInnerHTML={{
__html: fs.readFileSync(
Expand Down
4 changes: 2 additions & 2 deletions packages/gatsby/src/utils/webpack-utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -430,8 +430,8 @@ module.exports = async ({
*/
plugins.extractText = options =>
new MiniCssExtractPlugin({
filename: `../css/[name].[contenthash].css`,
chunkFilename: `../css/[name].[contenthash].css`,
filename: `[name].[contenthash].css`,
Copy link
Contributor Author

@brotzky brotzky Jul 12, 2018

Choose a reason for hiding this comment

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

I've added back the ../css/ back as this wasn't supposed to be in this commit. See the following commit for reference.

chunkFilename: `[name].[contenthash].css`,
...options,
})

Expand Down