Skip to content

NXT-570: Added background option for transparent Image - #405

Merged
dan-ichim-lgp merged 9 commits into
developfrom
feature/NXT-570
Jul 31, 2026
Merged

NXT-570: Added background option for transparent Image#405
dan-ichim-lgp merged 9 commits into
developfrom
feature/NXT-570

Conversation

@alexandrumorariu

Copy link
Copy Markdown
Contributor

Checklist

  • I have read and understand the contribution guide
  • A CHANGELOG entry is included
  • At least one test case is included for this feature or bug fix
  • I have run automated testing and it is passed
  • Documentation was added or is not needed
  • This is an API breaking change

Issue Resolved / Feature Added

Added storybook controls for background change in Image and ImageItem.

Resolution

Additional Considerations

Links

NXT-570

Comments

… and added Changed ImageItem to be able to use the new props.
Comment thread .github/workflows/ci-reusable.yml Outdated
@codecov

codecov Bot commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.66%. Comparing base (096891a) to head (7e69187).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #405   +/-   ##
========================================
  Coverage    82.66%   82.66%           
========================================
  Files          153      153           
  Lines         7620     7620           
  Branches      2318     2318           
========================================
  Hits          6299     6299           
  Misses        1027     1027           
  Partials       294      294           

☔ 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.

# Conflicts:
#	samples/sampler/stories/default/Image.js
#	samples/sampler/stories/default/ImageItem.js
Comment thread ImageItem/ImageItem.js Outdated
Comment thread ImageItem/ImageItem.js Outdated

@daniel-stoian-lgp daniel-stoian-lgp 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.

check my comments

alexandrumorariu and others added 2 commits July 30, 2026 13:11
Co-authored-by: Daniel Stoian <63335068+daniel-stoian-lgp@users.noreply.github.com>

@daniel-stoian-lgp daniel-stoian-lgp 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.

please add changelog. i will keep the PR blocked to avoid accidental merge of github actions file

Comment thread .github/workflows/ci-reusable.yml Outdated
@dan-ichim-lgp
dan-ichim-lgp merged commit 38b16f9 into develop Jul 31, 2026
6 checks passed
@dan-ichim-lgp
dan-ichim-lgp deleted the feature/NXT-570 branch July 31, 2026 11:11
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