Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 12, 2026

Provided design feedback on the allLayers configuration option name in response to maintainer question.

Context

The allLayers option was added to OutputCssLayersOptions to force output of all CSS layer names in the @layer declaration, even if some layers contain no rules:

{
  outputToCssLayers: {
    allLayers: true  // Outputs: @layer theme, preflights, unused-layer, default;
  }
}

Response

Analyzed the naming in context of:

  • Parent option name (outputToCssLayers)
  • Existing boolean option naming patterns in codebase
  • Core behavior (including unused/empty layers)

Recommended keeping allLayers for clarity and conciseness, with includeUnused as viable alternative if emphasizing the key differentiator.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add allLayers option to output all layer names Respond to feedback on allLayers option naming Jan 12, 2026
Copilot AI requested a review from zyyv January 12, 2026 07:13
Base automatically changed from core-output-all-layernames to main January 12, 2026 07:15
@zyyv zyyv closed this Jan 12, 2026
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.

2 participants