Skip to content

feat: NX2 - support custom background images and dark scheme on da-sites#870

Merged
auniverseaway merged 4 commits intomainfrom
dasites
Apr 29, 2026
Merged

feat: NX2 - support custom background images and dark scheme on da-sites#870
auniverseaway merged 4 commits intomainfrom
dasites

Conversation

@auniverseaway
Copy link
Copy Markdown
Member

  • Pass background element from browse block into da-sites component
  • Replace hardcoded hex colors with --s2-* CSS variables
  • Add gradient mask on background image
  • Set color scheme from localStorage/media query before render
  • Load utils.js early and hide HTML to prevent FOUC
  • Select LCP images based on active color scheme

How Has This Been Tested?

DA.LIVE prod + NX1 + old images

https://dasites--da-live--adobe.aem.live/
Expectation: everything works E2E

DA.LIVE stage + NX2 + new images

https://dasites--da-live--adobe.aem.page/?nx=nx2fixes&nxver=2
Expectation: only da homepage works, but is correctly styled

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • [] My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@aem-code-sync
Copy link
Copy Markdown

aem-code-sync Bot commented Apr 4, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

- Pass background element from browse block into da-sites component
- Replace hardcoded hex colors with --s2-* CSS variables
- Add gradient mask on background image
- Set color scheme from localStorage/media query before render
- Load utils.js early and hide HTML to prevent FOUC
- Select LCP images based on active color scheme
hannessolo
hannessolo previously approved these changes Apr 29, 2026
@aem-code-sync aem-code-sync Bot temporarily deployed to dasites April 29, 2026 16:17 Inactive
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@auniverseaway auniverseaway merged commit 49b0493 into main Apr 29, 2026
3 of 4 checks passed
@auniverseaway auniverseaway deleted the dasites branch April 29, 2026 17:14
chrischrischris pushed a commit that referenced this pull request May 1, 2026
…tes (#870)

* feat: nx2 + browse

- Pass background element from browse block into da-sites component
- Replace hardcoded hex colors with --s2-* CSS variables
- Add gradient mask on background image
- Set color scheme from localStorage/media query before render
- Load utils.js early and hide HTML to prevent FOUC
- Select LCP images based on active color scheme

* fix test and package-lock

* fix: Use Node.js 22 instead of non-existent version 24

* fix: Reset package-lock.json to main branch version

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.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.

3 participants