Skip to content

Refactor screen recording feature#1

Open
farhan-momin wants to merge 1 commit intomasterfrom
refactor-screen-record
Open

Refactor screen recording feature#1
farhan-momin wants to merge 1 commit intomasterfrom
refactor-screen-record

Conversation

@farhan-momin
Copy link
Owner

This PR resolves sugarlabs#4505

Changes Made to the functionality

  • Screen sharing requires permission only once for recording multiple clips.
  • Stop sharing does not need to be clicked every time for a recording.

Working of Record feature after changes

  • Clicking on Record button MB asks for screen share permission.
  • Allowing permission shares the current tab with MB and starts recording.
  • Multiple clips can be recorded with record button.
  • After stop sharing permission, above process can repeat.

Past Discussion about these changes can be found at PR sugarlabs#4506

working_after_changes_2.webm

@github-actions
Copy link

github-actions bot commented Mar 7, 2026

✅ All Jest tests passed! This PR is ready to merge.

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.

Improve screen recording working and fix repeated permission prompts

1 participant