Skip to content

(feat) GNL-804: Add icons to native home dashboard links - #2637

Open
ekemboi29 wants to merge 3 commits into
openmrs:mainfrom
ekemboi29:task/gnl-804-native-home-dashboard-icons
Open

ekemboi29 wants to merge 3 commits into
openmrs:mainfrom
ekemboi29:task/gnl-804-native-home-dashboard-icons

Conversation

@ekemboi29

Copy link
Copy Markdown

Requirements

  • This PR has a conventional title that includes the implementation ticket number.
  • This follows the existing OpenMRS home dashboard icon pattern; no new design is introduced.
  • The change is validated by the repository pre-push checks.

Summary

Adds standard OpenMRS icons to the existing native home dashboard links for Appointments, Service queues, Patient lists, and Wards. This lets distributions use the upstream links directly without registering replacement links solely to provide iconography.

The icons are added through each app’s existing dashboardMeta object:

  • Appointments: omrs-icon-calendar
  • Service queues: omrs-icon-message-queue
  • Patient lists: omrs-icon-group
  • Wards: omrs-icon-hospital-bed

Screenshots

Not attached. This declarative metadata change uses the existing DashboardExtension rendering and standard OpenMRS icons without changing layout or interaction.

Related Issue

GNL-804 — Cleanup the home page iconography (downstream implementation issue).

Other

All 27 repository pre-push lint, TypeScript, and test tasks passed.

@ekemboi29 ekemboi29 changed the title feat: add icons to native home dashboard links (GNL-804) (feat) GNL-804: Add icons to native home dashboard links Jul 24, 2026

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

Thanks for the good work @ekemboi29 ,
Clean and minimal — nice use of the existing framework icon prop. Two small asks:

Could you add a screenshot showing the icons rendering on the home page?
Should appointmentCalendarDashboardMeta also get an icon, or is it not a home link?

@EDSONZ-WASSWA

Copy link
Copy Markdown
Contributor

Hey @ekemboi29 Really nice work here. This is exactly the kind of change that makes it cool like ..small, focused, and solves a real problem.
I like how you thought this through. Adding the icons declaratively means distributions can actually use these out-of-the-box now without having to override them, and that's genuinely useful. The icon choices make sense too calendar for appointments, a queue icon for service queues, that kind of thing just works.
Also the consistency across all four modules is perfect, ..no overthinking, just a solid improvement to the codebase.

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.

3 participants