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.

Sign up for GitHub

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

Jump to bottom

892307-Update theme support in surface chart documents. #1476

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

892307-Update theme support in surface chart documents. #1476

wants to merge 2 commits into from

Conversation

sanjayvikramsf4373
Copy link
Contributor

@sanjayvikramsf4373 sanjayvikramsf4373 commented Aug 13, 2024 •
edited
Loading

Description

  • Update the UG for surface chart new theme support.
  • Removed dependent assemblies from control dependencies for scheduler (Themes MaterialLight) and Gantt (Themes MaterialLight & SfSkinManager).

@SyncfusionBuild
Copy link
Contributor

Build Status: INPROGRESS 🔃
Build Location: 892307-Update_Surface_chart_document

@SyncfusionBuild
Copy link
Contributor

CI Status: SUCCESS ✅
Source Compilation: SUCCESS ✅
Build Location: 892307-Update_Surface_chart_document
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

@SyncfusionBuild
Copy link
Contributor

Build Status: INQUEUE 🕒
Build Location: 892307-Update_Surface_chart_document

@SyncfusionBuild
Copy link
Contributor

Build Status: INPROGRESS 🔃
Build Location: 892307-Update_Surface_chart_document

@SyncfusionBuild
Copy link
Contributor

CI Status: SUCCESS ✅
Source Compilation: SUCCESS ✅
Build Location: 892307-Update_Surface_chart_document
Technical Error(s): 0
Spelling Error(s): 0
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

Karthickmani97
Karthickmani97 reviewed Aug 28, 2024
View reviewed changes
wpf/Surface-Chart/Getting-Started.md
@@ -516,3 +516,12 @@ The following output is displayed as a result of the above code example.
You can get the complete getting started sample [`here`](https://www.syncfusion.com/downloads/support/directtrac/general/ze/SurfaceChartDemo-1998279060).

![simple surface from Code behind](surface_chart_images/surface_chart_img7.jpeg)

## Theme
The SfSurfaceChart supports various built-in themes. Refer to the below links to apply themes for the SfSurfaceChart,
Copy link
Collaborator

Choose a reason for hiding this comment

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

WPF SurfaceChart (SfSurfaceChart) supports various built-in themes. Refer to the below links to apply themes for the SfSurfaceChart,

Karthickmani97
Karthickmani97 reviewed Aug 28, 2024
View reviewed changes
wpf/Surface-Chart/Getting-Started.md

* [Create a custom theme using ThemeStudio](https://help.syncfusion.com/wpf/themes/theme-studio#creating-custom-theme)

![Simple Surface](surface_chart_images/Theming_in_surfacechart.jpeg)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Change alt name and image name to theming-support-in-WPF-surface-chart-control

Karthickmani97
Karthickmani97 reviewed Aug 28, 2024
View reviewed changes
wpf/Surface-Chart/Getting-Started.md

* [Create a custom theme using ThemeStudio](https://help.syncfusion.com/wpf/themes/theme-studio#creating-custom-theme)

![Simple Surface](surface_chart_images/Theming_in_surfacechart.jpeg)
Copy link
Collaborator

Choose a reason for hiding this comment

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

As of now, Remove image , once updated surface chart UX review correction, and the share image.

Karthickmani97
Karthickmani97 reviewed Aug 28, 2024
View reviewed changes
wpf/Gantt/migration-to-26v-or-higher.md
Syncfusion.SfSkinManager.WPF<br/>
Syncfusion.Themes.MaterialLight.WPF
</td>
<td>Syncfusion.SfGrid.WPF</td>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Share PR in development

@Karthickmani97 Karthickmani97 mentioned this pull request Aug 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Karthickmani97 Karthickmani97 Karthickmani97 left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sanjayvikramsf4373 @SyncfusionBuild @Karthickmani97