Skip to content

chore(deps): upgrade focus-trap-react and node-sass-glob-importer in ui-components#1708

Merged
ArtieReus merged 3 commits into
mainfrom
artie-upgrade-focus-trap-and-node-sass
May 21, 2026
Merged

chore(deps): upgrade focus-trap-react and node-sass-glob-importer in ui-components#1708
ArtieReus merged 3 commits into
mainfrom
artie-upgrade-focus-trap-and-node-sass

Conversation

@ArtieReus

@ArtieReus ArtieReus commented May 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

Upgrades focus-trap-react from 11.0.6 to 12.0.2 and node-sass-glob-importer from 3.0.2 to 5.3.3 in the ui-components package. Both are devDependencies used for development and Storybook builds.

Changes Made

  • Upgraded focus-trap-react from 11.0.6 to 12.0.2 in packages/ui-components/package.json
    • Used in Modal component for focus management
    • Breaking change: onPostActivate() timing corrected (not used in our codebase)
  • Upgraded node-sass-glob-importer from 3.0.2 to 5.3.3 in packages/ui-components/package.json
    • Used in Storybook config for SCSS preprocessing with glob syntax
    • Major version jump (3.x → 5.x) but API remains compatible
  • Updated pnpm-lock.yaml to reflect dependency changes

Related Issues

  • N/A - Routine dependency upgrades

Screenshots (if applicable)

N/A - No visual changes

Testing Instructions

  1. pnpm install
  2. cd packages/ui-components && pnpm build - Verify ui-components builds successfully
  3. cd packages/ui-components && pnpm build-storybook - Verify Storybook builds (tests node-sass-glob-importer)
  4. pnpm build - Verify all packages and apps build successfully
  5. Test Modal component functionality manually to verify focus-trap-react works correctly

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have made corresponding changes to the documentation (if applicable).
  • My changes generate no new warnings or errors.
  • I have created a changeset for my changes.

PR Manifesto

Review the PR Manifesto for best practises.

ArtieReus added 2 commits May 21, 2026 09:54
…porter to 5.3.3

Signed-off-by: Arturo Reuschenbach Puncernau <reuschenbach@gmail.com>
Signed-off-by: Arturo Reuschenbach Puncernau <reuschenbach@gmail.com>
@ArtieReus ArtieReus requested a review from franzheidl as a code owner May 21, 2026 07:57
Copilot AI review requested due to automatic review settings May 21, 2026 07:57
@ArtieReus ArtieReus requested a review from a team as a code owner May 21, 2026 07:57
@changeset-bot

changeset-bot Bot commented May 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 10948c3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@cloudoperators/juno-ui-components Patch
@cloudoperators/juno-app-carbon Patch
@cloudoperators/juno-app-doop Patch
@cloudoperators/juno-app-example Patch
@cloudoperators/juno-app-greenhouse Patch
@cloudoperators/juno-app-heureka Patch
@cloudoperators/juno-app-supernova Patch
@cloudoperators/juno-app-template Patch
@cloudoperators/juno-messages-provider Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

github-actions Bot commented May 21, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-05-21 15:05 UTC

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Upgrades devDependencies in @cloudoperators/juno-ui-components to keep focus-management (Modal) and SCSS glob importing (Storybook build) up to date, with corresponding lockfile updates and a patch changeset.

Changes:

  • Bumped focus-trap-react from 11.0.6 to 12.0.2.
  • Bumped node-sass-glob-importer from 3.0.2 to 5.3.3.
  • Added a changeset and updated pnpm-lock.yaml for the dependency graph changes.

Reviewed changes

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

File Description
packages/ui-components/package.json Updates the two devDependency versions for ui-components.
pnpm-lock.yaml Regenerates the lockfile to reflect the upgraded dependencies and new transitive deps.
.changeset/five-jeans-visit.md Adds a patch changeset documenting the dependency upgrades.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread .changeset/five-jeans-visit.md
Resolved pnpm-lock.yaml conflict by regenerating with pnpm install

Signed-off-by: Arturo Reuschenbach Puncernau <reuschenbach@gmail.com>

@franzheidl franzheidl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested in several Modals, works for me. 👍

@ArtieReus ArtieReus merged commit 374c671 into main May 21, 2026
25 checks passed
@ArtieReus ArtieReus deleted the artie-upgrade-focus-trap-and-node-sass branch May 21, 2026 15:05
hodanoori pushed a commit that referenced this pull request Jun 11, 2026
…ui-components (#1708)

* chore(deps): upgrade focus-trap-react to 12.0.2 and node-sass-glob-importer to 5.3.3

Signed-off-by: Arturo Reuschenbach Puncernau <reuschenbach@gmail.com>

* chore(ui): added changeset

Signed-off-by: Arturo Reuschenbach Puncernau <reuschenbach@gmail.com>

---------

Signed-off-by: Arturo Reuschenbach Puncernau <reuschenbach@gmail.com>
Signed-off-by: Hoda Noori <hoda.noori@sap.com>
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.

[Task] (Juno): Upgrade focus-trap-react from v11.x.x to v12.x.x [Task] (Juno): Upgrade node-sass-glob-importer from v3.x.x to v5.x.x

5 participants