Skip to content

RHDHBUGS-3359: Add missing Common icons reference section#2382

Open
rh-tokeefe wants to merge 3 commits into
redhat-developer:release-1.9from
rh-tokeefe:RHDHBUGS-3359-1.9
Open

RHDHBUGS-3359: Add missing Common icons reference section#2382
rh-tokeefe wants to merge 3 commits into
redhat-developer:release-1.9from
rh-tokeefe:RHDHBUGS-3359-1.9

Conversation

@rh-tokeefe

Copy link
Copy Markdown
Member

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s):
release 1.9

Issue:
https://redhat.atlassian.net/browse/RHDHBUGS-3359

Summary:
This PR fixes broken cross-references to a missing "Common icons for customization" reference section by creating the missing content.

Changes:

  • Created modules/shared/ref-common-icons-for-customization.adoc with a table of commonly used icon identifiers
  • Added the new reference module to the global header assembly
  • Documented alternative icon types (URLs, SVG paths, relative paths)
  • Linked to the upstream Backstage icon set for complete documentation

Fixes broken links in:

  • proc-configure-the-logo-in-the-global-header.adoc
  • proc-customize-your-rhdh-quick-start.adoc
  • proc-use-hosted-json-files-to-provide-data-to-the-quick-access-card.adoc

Preview:
Will be available after build completes

@rhdh-bot

rhdh-bot commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

PR Build Results

Build failed -- 33/34 titles | 71s
Preview: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-2382/ (stale -- title build failed, showing previous version)

configure_customizing-rhdh

Error: /docs/titles/configure_customizing-rhdh/build/en-US/master.xml fails to validate
Cause: Document structure validation failed
Fix: Run 'ccutil compile' locally and check the full error output

View full logs


Content Quality Assessment

CQA Report

  • CQA-00a: Orphaned modules
  • CQA-00b: Directory structure
  • CQA-01: Vale AsciiDoc DITA compliance
  • CQA-02: Verify assembly structure
  • CQA-03: Verify content type metadata
  • CQA-04: Verify module templates
  • CQA-05: Verify required modular elements
  • CQA-06: Verify assemblies follow official template (one user story)
  • CQA-07: Verify TOC depth (max 3 levels)
  • CQA-08: Verify short description content quality
  • CQA-09: Verify short description format
  • CQA-10: Verify titles are brief, complete, and descriptive
  • CQA-11: Verify procedure prerequisites
  • CQA-12: Verify grammar and style (Vale)
  • CQA-13: Verify content matches declared type
  • CQA-14: Verify no broken links
  • CQA-15: Check redirects
  • CQA-16: Verify official product names
  • CQA-17: Verify legal disclaimers for preview features

Summary

Checks: 19 total, 19 pass, 0 fail

19 checks: 19 pass, 0 fail

Run node build/scripts/cqa/index.js --all --fix locally to review and auto-fix issues.


Updated 2026-06-16 18:27:12 UTC

Created ref-common-icons-for-customization.adoc to document the
predefined system icon identifiers available in RHDH. This fixes
broken cross-references in three procedure files that were linking
to a non-existent #ref-common-icons anchor.

The new reference module:
- Lists commonly used icon identifiers (add, catalog, github, etc.)
- Explains alternative icon types (URLs, SVG paths, relative paths)
- Links to extend-internal-icon-catalog for custom icons
- Provides link to upstream Backstage icon set

Also updated assembly-configure-the-global-header-in-rhdh.adoc to
include the new reference module.

Fixes broken links in:
- proc-configure-the-logo-in-the-global-header.adoc
- proc-customize-your-rhdh-quick-start.adoc
- proc-use-hosted-json-files-to-provide-data-to-the-quick-access-card.adoc

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Comment thread modules/shared/ref-common-icons-for-customization.adoc Outdated
Comment thread modules/shared/ref-common-icons-for-customization.adoc Outdated
Updated xref to extend-internal-icon-catalog to use the actual
context value 'front-end-plugin-wiring' instead of the {context}
variable, following documentation best practices.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Removed the link to the upstream Backstage GitHub repository to
comply with product documentation guidelines that avoid linking
to external GitHub repositories.

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

Copy link
Copy Markdown

@dzemanov

dzemanov commented Jun 17, 2026

Copy link
Copy Markdown
Member

I believe icons available by default are located here, we should probably mention all of them?


[NOTE]
====
In addition to these predefined icon identifiers, you can use:

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.

I think it would make sense to also mention Backstage system icons that are available.

====
In addition to these predefined icon identifiers, you can use:

* *Full image URLs*: External image URLs (for example, `https://example.com/icon.png`)

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.

Can you also add Data encoded URIs?

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.


include::../modules/shared/proc-enable-quicklinks-and-starred-items-after-an-upgrade.adoc[leveloffset=+1]

include::../modules/shared/ref-common-icons-for-customization.adoc[leveloffset=+1]

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.

Can you add the link for common icons also to scorecard icons section?

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.

I think quickstart plugin has also reference to common icons, can you check if it is working and if there are any other non-working common icons references and update them to this reference? Thank you!

@dzemanov dzemanov Jun 17, 2026

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants