Skip to content

Optional Icon Block integration (dual source, preset grid preserved) - #5

Open
jrevillini wants to merge 1 commit into
ndiego:mainfrom
jrevillini:feature/icon-block-integration
Open

Optional Icon Block integration (dual source, preset grid preserved)#5
jrevillini wants to merge 1 commit into
ndiego:mainfrom
jrevillini:feature/icon-block-integration

Conversation

@jrevillini

@jrevillini jrevillini commented May 23, 2026

Copy link
Copy Markdown

Summary

Adds an optional The Icon Block bridge for core/button:

  • Toolbar bolt reuses Icon Block inserter/modals (@icon-block at build time).
  • Sidebar keeps the original preset grid; adds a compact ICON SOURCE section (Built-in Library vs The Icon Block + Reset).
  • Dual storage: preset slug and library SVG can coexist; iconSource selects display.
  • Front end renders saved ebiIcon when Icon Block is not active.

Relates to #2 — happy to collaborate or supersede that branch if this direction fits better.

Why this vs #2

#2 (existing) This PR
Preset grid when Icon Block active Replaced / hidden Always visible
Source switching Toolbar-centric iconSource + sidebar Reset
Back-compat Explicit preset + library retention

Documentation

  • Architecture / traceability: docs/ARCHITECTURE.md
  • Manual tests: docs/ICON_SOURCE_TEST_CHECKLIST.md
  • Build: clone icon-block sibling, npm install && npm run build (see README)

Demo

Test plan

  • Preset pick → bolt library icon → Reset → preset highlighted + canvas match
  • Placement toggles work for preset and library
  • Deactivate Icon Block: front still shows ebiIcon; editor warning; presets work
  • Legacy posts with only icon slug unchanged
  • npm run build clean with Icon Block clone at ../icon-block

Diff notes

20 file pairs differ (see manifest.txt)


Branch: feature/icon-block-integration · Generated 2026-05-23T11:54:53Z

Preserve preset grid; dual iconSource storage; toolbar bolt picker.
See docs/ARCHITECTURE.md and docs/ICON_SOURCE_TEST_CHECKLIST.md.
@jrevillini

jrevillini commented May 23, 2026

Copy link
Copy Markdown
Author

Discussion started in #4 — relates to #2. @ndiego when you have a moment, demo video is in the PR description; live demo site available on request. Happy to adjust based on your preferred direction.

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.

1 participant