Skip to content

add material icons stylesheet link in basic catalog index#54

Open
jgindin wants to merge 1 commit into
mainfrom
fix-material-icons
Open

add material icons stylesheet link in basic catalog index#54
jgindin wants to merge 1 commit into
mainfrom
fix-material-icons

Conversation

@jgindin

@jgindin jgindin commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Description

Enables correct rendering of material icons inside the sandboxed preview iframe running the ng-basic-catalog app.

Pre-launch Checklist

  • I signed the [CLA].
  • I read the [Contributors Guide].
  • I read the [Style Guide].
  • I have added updates to the [CHANGELOG].
  • I updated/added relevant documentation.
  • My code changes (if any) have tests.
  • If my branch is on fork, I have verified that scripts/e2e_test.sh passes.

@jgindin jgindin requested a review from sugoi-yuzuru June 19, 2026 20:12

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds the Material Icons stylesheet link to the index.html file of the ng-basic-catalog sample. The reviewer suggested optimizing the loading performance of this stylesheet by adding preconnect resource hints for the Google Fonts domains.

Comment thread samples/ng-basic-catalog/src/index.html
@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

⚡ A2UI Composer PR Preview

Your automated preview is successfully live:
👉 Launch PR Preview

Note: This environment will be wiped automatically when the PR is merged or closed.

Enables correct rendering of material icons inside the sandboxed preview
iframe running the ng-basic-catalog app.
@jgindin jgindin force-pushed the fix-material-icons branch from adfd139 to d7903f4 Compare June 19, 2026 20:18
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