Skip to content

FLUT-965284 - [Feature] Added Sliders UG improvement changes in hotfix branch. #1292

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

Conversation

sfHariHaraSudhan
Copy link
Collaborator

Feature description

Review and update the API documentation for the slider, range slider and range selector widgets.

Key objectives:

  • Ensure all public APIs are fully documented.
  • Ensure the inclusion of correct hyperlinks, default property values in documentation.
  • Ensure the public APIs content were correct in the documentation.

Analysis and design

  • Task ID - [FLUT-965284]
  • Audit all public APIs in the sliders widget to ensure comprehensive documentation.
  • Identify any undocumented APIs and draft descriptive content, including usage examples.
  • Verify that all API entries contain appropriate hyperlinks for seamless developer reference.
  • Confirm that default property values are clearly specified across all relevant entries.
  • Detect and correct inaccuracies or misleading information in the existing documentation.
  • Issue noted document link - Sliders UG documentation issues

Solution description

  • Verified the full set of APIs by cross-checking with the API reference on pub.dev.
  • Identified missing APIs and added comprehensive documentation for them.
  • Reviewed and updated the documentation to include any missing hyperlinks for accurate cross-referencing.
  • Assessed the API documentation content for accuracy and made necessary corrections.
  • Validated default property values and updated them where discrepancies were found.

Output screenshots

edgeLabelPlacement

Range slider:
image
image

Slider:
image
image

Range Selector:
image

Areas affected and ensured

Code snippets and content have been affected due to the modification.

API Changes

Missed API in this documentation changes

  • edgeLabelPlacement - Added
  • overlayShape - Not Needed, We can directly customize from slider theme data

Behavioral Changes

No behavioral changes due to this modification.

Test cases

No test cases for this PR.

Testbed sample location

No needed for this specific task.

Does it have any known issues?

No.

Does it have memory leak?

Not applicable

MR CheckList

  • Does it follow the design guidelines? It is mandatory that, we should not move the changes without reading this. Yes, followed the guidelines
  • Did UI automation passed without errors? If it is customer issue, make sure it is included in the IR automation. It is a documentation related work, so Not applicable
  • Properly working in Xamarin.Forms previewer.
  • Ensured in iOS, Android, UWP and macOS (if supported). Not applicable
  • Have you added API comments if it is new API? Not applicable
  • Does it need localization? If so, update resx file in GitHub. Not applicable
  • Whether we have added new APIs? If so, whether it is as per the guidelines? Not applicable
  • If changes are made in the Xamarin.Forms sample browser project file, whether it is included in the props file too? If not, your sample level changes will not reflect in the application published in the hockey app. Not applicable
  • Have you ensured the changes in Android API 19 and iOS 9? Not applicable

Copy link
Collaborator Author

@sfHariHaraSudhan sfHariHaraSudhan left a comment

Choose a reason for hiding this comment

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

All changes are fine.
Dev PR: 1274

@SyncfusionBuild
Copy link
Contributor

Your pull request has been created. Syncfusion support team will validate this commit and publish your changes accordingly.

Copy link
Collaborator

@VijayakumarMariappan VijayakumarMariappan 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

@VijayakumarMariappan VijayakumarMariappan merged commit ffe63db into hotfix/hotfix-v30.1.37 Jul 30, 2025
1 check passed
@VijayakumarMariappan VijayakumarMariappan deleted the FLUT-965284-Slider-ug-hotfix branch July 30, 2025 03:47
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.

3 participants