Skip to content

chore(icons): add a help icon svg partial - #7470

Merged
mcoker merged 8 commits into
patternfly:mainfrom
srambach:7265-help-icon-partial
Apr 22, 2025
Merged

mcoker merged 8 commits into
patternfly:mainfrom
srambach:7265-help-icon-partial

Conversation

@srambach

Copy link
Copy Markdown
Member

Fixes #7265

What

Adds a new help icon partial in the assets/images directory and updates several components to use it instead of Font Awesome's question-circle icon.

Changes

  • Added icon-help.hbs to src/patternfly/assets/images/ containing an SVG help icon
  • Updated components to use the new icon:
    • Table column help action
    • Modal box header help
    • Code editor keyboard shortcuts
    • Button example (plain with no padding)

How

The icon is implemented as a handlebars partial that can be referenced using button--icon-template='assets/images/icon-help'.

Note

I left the filled icon in the masthead and in the input group because those are standalone-ish buttons.

@patternfly-build

patternfly-build commented Apr 17, 2025 •

Copy link
Copy Markdown
Collaborator

@andrew-ronaldson

Copy link
Copy Markdown
Collaborator

Hey hey. It looks like the forms are using Outlined Question Circle instead of help. I realize now my screenshot was misleading as it looks more like the outlined version. My apologies!
It is good in the tabs examples.

@andrew-ronaldson andrew-ronaldson left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

❓ ✅

@mcoker mcoker 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.

LGTM though if it's no sweat, it would be good to remove the path from the icon-help partial name.

Comment thread src/patternfly/components/Button/examples/Button.md Outdated
Comment thread src/patternfly/components/CodeEditor/examples/CodeEditor.md Outdated
Comment thread src/patternfly/components/CodeEditor/examples/CodeEditor.md Outdated
Comment thread src/patternfly/components/Form/form-group-label-help.hbs Outdated
Comment thread src/patternfly/components/ModalBox/modal-box-header-help.hbs Outdated
Comment thread src/patternfly/components/Table/table-column-help-action.hbs Outdated

@mcoker mcoker 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.

🚀

@mcoker
mcoker merged commit 7855d70 into patternfly:main Apr 22, 2025
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.

Form: Update examples to use <HelpIcon>

4 participants