Skip to content

896838 - Publish a GitHub sample for theming the PDF Viewer using theme keys #54

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

Merged
merged 4 commits into from
May 21, 2025

Conversation

Bharathi4332
Copy link
Collaborator

Created a sample project that demonstrates how to apply themes to the PDF Viewer using theme keys. The sample includes updates for contrast themes where the hover and pressed states of buttons and list views are set to aquatic colors.

@@ -0,0 +1,457 @@
<?xml version = "1.0" encoding = "UTF-8" ?>
Copy link
Collaborator

Choose a reason for hiding this comment

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

why do we use three documents/ if it necessary for theme can we load form document by default and remove pdf succincly/

image

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have attached only "PDF_Succinctly" PDF and removed the other PDFs

@@ -0,0 +1,457 @@
<?xml version = "1.0" encoding = "UTF-8" ?>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have changed the sample inside the Styles folder

{
InitializeComponent();
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

avoid unwanted new lines

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have removed the unwanted lines


namespace PDFViewerThemes
{
class PdfViewerViewModel : INotifyPropertyChanged
Copy link
Collaborator

Choose a reason for hiding this comment

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

image

does this not showing any error or warning when you accessing the private class in mainpage.xaml/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No error or warning occurs. please find the screenshot
Screenshot 2025-05-21 201457

{
class PdfViewerViewModel : INotifyPropertyChanged
{
private Stream? pdfDocumentStream;
Copy link
Collaborator

Choose a reason for hiding this comment

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

image

this may required system.io namespace. does it not shown any warning or error/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No error or warning occurs. please find the screenshot
Screenshot 2025-05-21 201457

2. Syncfusion.Maui.PdfViewer NuGet package.

## Steps to Apply Themes

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please mention UG link for themes in the step

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have added the UG link for themes

@Deepak1799 Deepak1799 merged commit da95908 into master May 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants