Skip to content

feat: add configuration for postcss prefixwrap ignored selectors#120

Merged
Yannicvanveen merged 4 commits into
mainfrom
feat/prefixwrap-ignored-selectors-config-option
May 7, 2026
Merged

feat: add configuration for postcss prefixwrap ignored selectors#120
Yannicvanveen merged 4 commits into
mainfrom
feat/prefixwrap-ignored-selectors-config-option

Conversation

@Yannicvanveen
Copy link
Copy Markdown
Contributor

@Yannicvanveen Yannicvanveen commented May 7, 2026

Maakt de ignored selector lijst van de postcss prefix wrapper uitbreidbaar vanuit de Vite config van een project.

export default braveConfig( {
	editorStylesPrefixWrap: {
		entryPoints: [ 'resources/styles/editor.css' ],
		additionalIgnoredSelectors: [
			/^html.+$/
		],
	},
} );

Copilot AI review requested due to automatic review settings May 7, 2026 08:06
@Yannicvanveen Yannicvanveen requested a review from a team as a code owner May 7, 2026 08:06
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the @yardinternet/vite-config Brave Vite preset so projects can configure which selectors are ignored by the PostCSS Prefix Wrap step (used for wrapping editor styles inside .editor-styles-wrapper), and updates the postcss-prefixwrap dependency version.

Changes:

  • Add DEFAULT_IGNORED_SELECTORS and support a new ignoredSelectors option in getPostCssPrefixWrapPlugin.
  • Refactor ignoredSelectors construction to use defaults + optional project-provided selectors.
  • Bump postcss-prefixwrap from ^1.57.2 to ^1.58.0.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
packages/vite-config/src/utils/get-postcss-prefixwrap-plugin.js Adds default ignored selectors constant and allows configuring ignored selectors via editorStylesPrefixWrap.
packages/vite-config/package.json Updates postcss-prefixwrap dependency version.

Comment thread packages/vite-config/src/utils/get-postcss-prefixwrap-plugin.js
Comment on lines +19 to +21
const ignoredSelectors = Array.isArray( config.ignoredSelectors )
? [ ...DEFAULT_IGNORED_SELECTORS, ...config.ignoredSelectors ]
: DEFAULT_IGNORED_SELECTORS;
Comment thread packages/vite-config/package.json
@Yannicvanveen Yannicvanveen merged commit 743e456 into main May 7, 2026
1 check passed
@Yannicvanveen Yannicvanveen deleted the feat/prefixwrap-ignored-selectors-config-option branch May 7, 2026 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants