Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(PdfJS): move the PdfMobileViewer component to a standalone project #2335

Merged
merged 4 commits into from
Feb 27, 2025

Conversation

capdiem
Copy link
Contributor

@capdiem capdiem commented Feb 26, 2025

Pull Request Description

Title: refactor(PdfJS): move the PdfMobileViewer component to a standalone project.

Description:
This pull request refactors the PdfJS module by moving the PdfMobileViewer component to a standalone project. This change aims to improve the modularity and maintainability of the codebase, allowing for better separation of concerns and easier future enhancements.

Changes Made:

  • The PdfMobileViewer component has been extracted from the existing PdfJS module.
  • Updated references and imports to ensure the component functions correctly in its new standalone context.

This refactor is a step towards a more organized project structure, facilitating easier updates and potential reuse of the PdfMobileViewer component in other parts of the application or in different projects.

Please review the changes and provide feedback. Thank you!

@capdiem capdiem changed the title refactor(PdfJS): move the PdfMobileViewer component to a standalone project. refactor(PdfJS): move the PdfMobileViewer component to a standalone project Feb 26, 2025
@capdiem capdiem requested a review from Copilot February 26, 2025 09:20
Copy link

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

Copilot reviewed 21 out of 21 changed files in this pull request and generated 1 comment.

capdiem and others added 3 commits February 26, 2025 17:22
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Qinyouzeng Qinyouzeng merged commit b705aba into main Feb 27, 2025
1 check passed
@Qinyouzeng Qinyouzeng deleted the move-pdf-mobile-viewer branch February 27, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants