Skip to content

fix(back-to-top): change background color on hover and focus#3096

Merged
bennypowers merged 4 commits intomainfrom
fix/back-to-top-hover
Apr 15, 2026
Merged

fix(back-to-top): change background color on hover and focus#3096
bennypowers merged 4 commits intomainfrom
fix/back-to-top-hover

Conversation

@bennypowers
Copy link
Copy Markdown
Member

Summary

Apply hover/focus background color directly on the <a> element rather
than through CSS custom property overrides that don't cascade into
pf-button's shadow DOM. Also adds pf-button hover/focus-within styles.

Closes #2743

Test plan

  • Verify back-to-top button changes background on hover
  • Verify back-to-top button changes background on focus

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 15, 2026

🦋 Changeset detected

Latest commit: 684117d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@patternfly/elements Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 15, 2026

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit 3816169
😎 Deploy Preview https://deploy-preview-3096--patternfly-elements.netlify.app/

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026

✅ Commitlint tests passed!

More Info
{
  "valid": true,
  "errors": [],
  "warnings": [],
  "input": "fix(back-to-top): change background color on hover and focus"
}

@github-actions github-actions bot added the AT passed Automated testing has passed label Apr 15, 2026
@github-actions
Copy link
Copy Markdown
Contributor

SSR Test Run for 9686535: Report

Apply hover/focus colors directly rather than through CSS custom
property overrides that don't cascade into shadow DOM.

Closes #2743

Assisted-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bennypowers bennypowers force-pushed the fix/back-to-top-hover branch from d7ea9eb to c03a886 Compare April 15, 2026 12:48
@github-actions
Copy link
Copy Markdown
Contributor

SSR Test Run for 95c2f4c: Report

@github-actions

This comment has been minimized.

@bennypowers bennypowers enabled auto-merge (squash) April 15, 2026 13:08
@github-actions
Copy link
Copy Markdown
Contributor

SSR Test Run for d3a7d17: Report

@bennypowers bennypowers merged commit cc84a12 into main Apr 15, 2026
15 checks passed
@bennypowers bennypowers deleted the fix/back-to-top-hover branch April 15, 2026 13:40
@github-actions
Copy link
Copy Markdown
Contributor

SSR Test Run for 3816169: Report

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

Labels

AT passed Automated testing has passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pf-back-to-top: should change background color on hover and focus

2 participants