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

Adding docs around the new "css" Encore functions and disabling file tracking #13652

Closed
wants to merge 1 commit into from

Conversation

weaverryan
Copy link
Member

Hi!

Docs for symfony/webpack-encore-bundle#91

Cheers!

Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

Do we need a versionadded directive?

{{ encore_entry_js_source('my_entry')|raw }}
</script>

If you can't use these `encore_entry_*_source` functions, you can instead
Copy link
Contributor

Choose a reason for hiding this comment

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

Double backticks please

@@ -69,6 +69,7 @@ Guides

* :doc:`Using Bootstrap CSS & JS </frontend/encore/bootstrap>`
* :doc:`Creating Page-Specific CSS/JS </frontend/encore/page-specific-assets>`
* :doc:`Rendering Multiple Templates in a Request: Emails, PDFs </frontend/encore/file-tracking>`
Copy link
Member

Choose a reason for hiding this comment

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

Why aren't we doing :doc:`/frontend/encore/file-tracking` here? :)

{{ encore_entry_script_tags('entry1') }}
{% do encore_enable_file_tracking() %}

With this, *all* JS and CSS files for `entry1` will be rendered and
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
With this, *all* JS and CSS files for `entry1` will be rendered and
With this, *all* JS and CSS files for ``entry1`` will be rendered and

bundle re-renders CSS or JS files that it previously rendered. For
example, in a controller::


Copy link
Member

Choose a reason for hiding this comment

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

Suggested change



// src/Controller/SomeController.php

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
namespace App\Controller;

public function index(EntrypointLookupInterface $entrypointLookup)
{
$entrypointLookup->reset();
// render a template
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// render a template
// ... render a template

// render a template

$entrypointLookup->reset();
// render another template
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// render another template
// ... render another template

@wouterj wouterj added the Waiting Code Merge Docs for features pending to be merged label Nov 22, 2020
@javiereguiluz
Copy link
Member

Let's close this old pending PR because we're moving away from Webpack Encore very fast so it doesn't make sense to try to add new complex features like this one. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review Waiting Code Merge Docs for features pending to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants