Skip to content

RDKUI-882 Green tick indication missing on app tile after successful app installation#227

Open
suryag23 wants to merge 1 commit into
developfrom
882
Open

RDKUI-882 Green tick indication missing on app tile after successful app installation#227
suryag23 wants to merge 1 commit into
developfrom
882

Conversation

@suryag23

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses RDKUI-882 by adding a green tick visual indicator on DAC app tiles after a successful installation, and bumps the RefUI version to 6.0.28 to reflect the change.

Changes:

  • Add tick overlay + tick mark elements to DAC app tiles and show them briefly on successful install completion.
  • Introduce a shared _showGreenTick() helper in DACAppMixin to drive the success indicator for both catalog and main-view tiles.
  • Bump RefUI version from 6.0.27 → 6.0.28 in package config and UI settings.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
bolt/package-configs/com.rdkcentral.refui.json Bumps application version metadata to 6.0.28.
accelerator-home-ui/src/items/DacAppItem.js Adds tick overlay/mark nodes and positions them within ImageWrapper so they scale with the tile.
accelerator-home-ui/src/items/AppCatalogItem.js Adds _showGreenTick() to the shared mixin and adds tick overlay/mark nodes to the catalog tile template.
accelerator-home-ui/settings.json Updates UI settings version to 6.0.28.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mhughesacn mhughesacn left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi @suryag23 : Where did this image come from? If not original to RDK, it needs further discussion.

@suryag23

Copy link
Copy Markdown
Contributor Author

HI @mhughesacn
I have generated this tick.png through the PowerPoint by own

@yashaswini-rk yashaswini-rk 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.

Looks good,Please merge after Martin approves.

@mhughesacn
mhughesacn dismissed their stale review July 17, 2026 12:08

Not relevant - original content (Thank you.)

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.

4 participants