Skip to content

fix(global-header): localize header search “All results” and clear control#3052

Merged
its-mitesh-kumar merged 1 commit into
redhat-developer:mainfrom
lokanandaprabhu:fix/global-header-search-i18n
May 6, 2026
Merged

fix(global-header): localize header search “All results” and clear control#3052
its-mitesh-kumar merged 1 commit into
redhat-developer:mainfrom
lokanandaprabhu:fix/global-header-search-i18n

Conversation

@lokanandaprabhu
Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

Fixes:

https://redhat.atlassian.net/browse/RHDHBUGS-3074

Improves internationalization for the global-header search autocomplete: the footer link to the full search page and the clear control now use the plugin translation system instead of hardcoded English, with strings added for German, Spanish, French, Italian, and Japanese.

Screenshot 2026-05-06 at 12 03 35 PM

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

Wire search autocomplete strings to plugin translations with locale bundles;
update SearchOption tests to use shared translation mocks; refresh API reports.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rhdh-gh-app
Copy link
Copy Markdown

rhdh-gh-app Bot commented May 6, 2026

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-global-header workspaces/global-header/plugins/global-header patch v1.21.3

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.78%. Comparing base (5e4d2b2) to head (bc5be9a).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3052   +/-   ##
=======================================
  Coverage   60.78%   60.78%           
=======================================
  Files        2067     2067           
  Lines       63873    63875    +2     
  Branches    16580    16580           
=======================================
+ Hits        38827    38829    +2     
  Misses      24793    24793           
  Partials      253      253           
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 5e4d2b2
ai-integrations 70.03% <ø> (ø) Carriedforward from 5e4d2b2
app-defaults 69.60% <ø> (ø) Carriedforward from 5e4d2b2
augment 69.36% <ø> (ø) Carriedforward from 5e4d2b2
bulk-import 72.44% <ø> (ø) Carriedforward from 5e4d2b2
cost-management 16.49% <ø> (ø) Carriedforward from 5e4d2b2
dcm 32.85% <ø> (ø) Carriedforward from 5e4d2b2
extensions 61.42% <ø> (ø) Carriedforward from 5e4d2b2
global-floating-action-button 73.75% <ø> (ø) Carriedforward from 5e4d2b2
global-header 61.68% <100.00%> (+0.05%) ⬆️
homepage 50.92% <ø> (ø) Carriedforward from 5e4d2b2
konflux 91.01% <ø> (ø) Carriedforward from 5e4d2b2
lightspeed 69.96% <ø> (ø) Carriedforward from 5e4d2b2
mcp-integrations 81.59% <ø> (ø) Carriedforward from 5e4d2b2
orchestrator 33.15% <ø> (ø) Carriedforward from 5e4d2b2
quickstart 62.64% <ø> (ø) Carriedforward from 5e4d2b2
sandbox 79.49% <ø> (ø) Carriedforward from 5e4d2b2
scorecard 83.61% <ø> (ø) Carriedforward from 5e4d2b2
theme 64.54% <ø> (ø) Carriedforward from 5e4d2b2
translations 8.49% <ø> (ø) Carriedforward from 5e4d2b2
x2a 82.09% <ø> (ø) Carriedforward from 5e4d2b2

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e4d2b2...bc5be9a. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@its-mitesh-kumar its-mitesh-kumar left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label May 6, 2026
@its-mitesh-kumar its-mitesh-kumar merged commit 7cf6d61 into redhat-developer:main May 6, 2026
60 checks passed
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.

2 participants