Skip to content

900945 Update the images for Polar Chart MAUI UG demo sample #5

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 5 commits into from
Aug 23, 2024

Conversation

tineshsf4652
Copy link
Collaborator

Description:
This pull request addresses two key updates in the MAUI demo sample Polar Chart:
1. Image Update: Replaces the existing Polar Chart image with a new version that includes corrected wind direction data labels.
2. ViewModel Update: Modifies the ViewModel used in the sample to reflect the corrected wind direction data.

Changes Made:

  1. Image Update:
    • Replaced Image: The outdated Polar Chart image has been replaced with a new image showing accurate wind direction data labels.
    • Documentation Update: Adjusted references and file paths in the demo documentation to point to the new image.
  2. Sample ViewModel Update:
    • Corrected Data: Updated the ViewModel to reflect accurate wind direction data, ensuring that the sample demonstrates the correct data representation.
    • Code Changes: Revised sample code to align with the corrected data, ensuring consistency between the visual representation and underlying data.

Updated Image Screenshot :
MAUI_polar_chartdemo

Copy link
Collaborator

@SaiyathAliFathima SaiyathAliFathima left a comment

Choose a reason for hiding this comment

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

changes are fine

Copy link
Collaborator

@Saravanan-Madhesh Saravanan-Madhesh left a comment

Choose a reason for hiding this comment

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

Fine

@Saravanan-Madhesh Saravanan-Madhesh merged commit 4463380 into master Aug 23, 2024
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.

4 participants