Skip to content

Tweak/minor changes#1056

Merged
KMchaudhary merged 4 commits into
mainfrom
tweak/minor-changes
Sep 8, 2025
Merged

Tweak/minor changes#1056
KMchaudhary merged 4 commits into
mainfrom
tweak/minor-changes

Conversation

@KMchaudhary
Copy link
Copy Markdown
Collaborator

  • Close the Bookmarks and Locked tab by default.
  • Set Show Branding option to true for all videos. It will tak the brand image from gloab setting if added.
  • Fix the progress bar jumping to 100% issue, on Restart migration button click.

Copilot AI review requested due to automatic review settings September 8, 2025 07:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 implements minor UI/UX improvements and fixes for the video editor and migration tools. The changes focus on improving default settings and fixing a progress bar behavior issue.

  • Enable branding icon by default for all videos using global branding settings
  • Close Bookmarks and Locked tabs by default for better initial UI state
  • Fix progress bar jumping to 100% issue by removing immediate fetch on restart
  • Update migration tool descriptions for better clarity

Reviewed Changes

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

Show a summary per file
File Description
pages/video-editor/redux/slice/videoSlice.js Enable branding icon by default
pages/tools/components/tabs/Migration/VimeoVideoMigration.jsx Remove immediate fetch call and improve description text
pages/tools/components/tabs/Migration/CoreVideoMigration.jsx Remove immediate fetch call and improve description text
pages/media-library/components/folder-tree/LockedTab.jsx Set locked tab to closed by default
pages/media-library/components/folder-tree/BookmarkTab.jsx Set bookmark tab to closed by default

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread pages/tools/components/tabs/Migration/VimeoVideoMigration.jsx
Comment thread pages/tools/components/tabs/Migration/CoreVideoMigration.jsx Outdated
@KMchaudhary
Copy link
Copy Markdown
Collaborator Author

Screenshot 2025-09-08@nayemDevs 12 43 51 PM

@nayemDevs could you please confirm the updated message on both the migration tools?

@KMchaudhary KMchaudhary merged commit ea77479 into main Sep 8, 2025
3 of 4 checks passed
@KMchaudhary KMchaudhary deleted the tweak/minor-changes branch September 8, 2025 07:46
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