Skip to content

feat!: remove style.css export - #2675

Merged
Antreesy merged 2 commits into
mainfrom
feat/inline-css-v8
Oct 26, 2026
Merged

Antreesy merged 2 commits into
mainfrom
feat/inline-css-v8

Conversation

@Antreesy

Copy link
Copy Markdown
Contributor
  • Reverts fix(build): restore @nextcloud/dialogs/style.css output #2672
  • Styles are imported automatically by the package entry points, so the ./style.css export is removed from package.json.
  • Update the README, apps no longer need to import @nextcloud/dialogs/style.css.
  • Add a breaking change note for v8 to the changelog.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (ClaudeCode:claude-opus-5-5)

@Antreesy Antreesy added this to the v8.0.0 milestone Oct 15, 2026
@codecov

codecov Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.74%. Comparing base (f248ea3) to head (d4a1359).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2675   +/-   ##
=======================================
  Coverage   61.74%   61.74%           
=======================================
  Files          15       15           
  Lines         481      481           
  Branches      104      103    -1     
=======================================
  Hits          297      297           
  Misses        165      165           
  Partials       19       19           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@susnux

susnux commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

👍

@Antreesy Antreesy mentioned this pull request Oct 26, 2026
@Antreesy
Antreesy marked this pull request as ready for review October 26, 2026 15:08
- Reverts ca2f0c6 for v8, styles are imported by the JS entries again (`inlineCSS`).
- `dist/style.css` is no longer generated, which is a breaking change for apps that import `@nextcloud/dialogs/style.css`.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- Styles are imported automatically by the package entry points, so the `./style.css` export is removed from `package.json`.
- Update the README, apps no longer need to import `@nextcloud/dialogs/style.css`.
- Add a breaking change note for v8 to the changelog.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy requested a review from susnux October 26, 2026 15:21
@Antreesy Antreesy self-assigned this Oct 26, 2026
@Antreesy Antreesy added 3. to review enhancement New feature or request labels Oct 26, 2026
@Antreesy
Antreesy merged commit 1f4aa1a into main Oct 26, 2026
13 checks passed
@Antreesy
Antreesy deleted the feat/inline-css-v8 branch October 26, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants