Skip to content

fix(cli): inline CSS sourcemaps in addition to JS sourcemaps#8377

Merged
jcesarmobile merged 2 commits intoionic-team:mainfrom
terencehonles:inline-css-sourcemaps
Mar 24, 2026
Merged

fix(cli): inline CSS sourcemaps in addition to JS sourcemaps#8377
jcesarmobile merged 2 commits intoionic-team:mainfrom
terencehonles:inline-css-sourcemaps

Conversation

@terencehonles
Copy link
Contributor

@terencehonles terencehonles commented Mar 4, 2026

Description

This change updates the cli copy's --inline handling to also inline the CSS sourcemaps and not just the JS sourcemaps.

reopening: #6526

Change Type

  • Fix
  • Feature
  • Refactor
  • Breaking Change
  • Documentation

Rationale / Problems Fixed

Allow CSS sourcemaps to be included properly

Tests or Reproductions

Screenshots / Media

Platforms Affected

  • Android
  • iOS
  • Web

Notes / Comments

This change updates the cli copy's `--inline` handling to also inline
the CSS sourcemaps and not just the JS sourcemaps.
@terencehonles terencehonles force-pushed the inline-css-sourcemaps branch from 5f0730e to b9bd18b Compare March 4, 2026 10:21
@jcesarmobile jcesarmobile changed the title feat(cli): inline CSS sourcemaps in addition to JS sourcemaps fix(cli): inline CSS sourcemaps in addition to JS sourcemaps Mar 23, 2026
@jcesarmobile
Copy link
Member

I'm considering this a fix instead of feat since the --inline description says

if true, all source maps will be inlined for easier debugging on mobile devices
since it says "all source maps", but it's not doing it for css files, I think it should and doing it is a fix

@jcesarmobile jcesarmobile merged commit 69476ab into ionic-team:main Mar 24, 2026
11 of 12 checks passed
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