Skip to content

RDKUI-838 [RDKE] After launching the app from UI, initial key press is not working#214

Merged
suryag23 merged 5 commits into
support/rdk7-mainfrom
feature/yt26-fix
Jun 26, 2026
Merged

RDKUI-838 [RDKE] After launching the app from UI, initial key press is not working#214
suryag23 merged 5 commits into
support/rdk7-mainfrom
feature/yt26-fix

Conversation

@suryag23

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 25, 2026 06:38
@suryag23 suryag23 requested a review from yashaswini-rk June 25, 2026 06:39

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 adjusts Miracast/app-lifecycle handling to avoid RefUI/ResidentApp being (re)launched or focused in ways that can interfere with input immediately after launching an app, particularly during Miracast playback.

Changes:

  • Track Miracast playback as the effective top-most app and reset Miracast globals on STOPPED.
  • Prevent ResidentApp relaunch while Miracast is in PLAYING state across multiple RDKShell event handlers and exitApp.
  • Add a null-guard before issuing miracast.stopRequest(...) on onLaunched, and bump UI version.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
accelerator-home-ui/src/App.js Updates Miracast state handling and gates ResidentApp relaunch while Miracast is PLAYING; adds null-guard for stopRequest.
accelerator-home-ui/src/api/AppApi.js Prevents ResidentApp relaunch in exitApp when Miracast is PLAYING.
accelerator-home-ui/settings.json Bumps platform version to 4.7.21.

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

Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js Outdated
Comment thread accelerator-home-ui/src/App.js Outdated
Comment thread accelerator-home-ui/src/App.js Outdated
Comment thread accelerator-home-ui/src/App.js Outdated
Comment thread accelerator-home-ui/src/App.js Outdated
Comment thread accelerator-home-ui/src/api/AppApi.js Outdated

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

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

Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js Outdated

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

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

Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js
Comment thread accelerator-home-ui/src/App.js

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

LGTM

@suryag23 suryag23 merged commit b8d0d9a into support/rdk7-main Jun 26, 2026
5 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants