Skip to content

FLUT-965284 - [Feature] Included a Sliders UG improvement changes. #1274

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

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

Conversation

BPraveenBalu
Copy link
Collaborator

@BPraveenBalu BPraveenBalu commented Jul 7, 2025

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:
edge-label-placement
vertical-edge-label-placement

Slider:
edge-label-placement
vertical-edge-label-placement

Range Selector:
selector-edge-label-placement

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

@SyncfusionBuild
Copy link
Contributor

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

@BPraveenBalu BPraveenBalu changed the title FLUT-965284 - [Others] Included a Sliders UG improvement changes. FLUT-965284 - [Feature] Included a Sliders UG improvement changes. Jul 8, 2025
Copy link
Collaborator

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

Address the suggested concerns.

@sfHariHaraSudhan
Copy link
Collaborator

Also add the output images in the PR Template for the newly included APIs in user guide.

@SyncfusionBuild
Copy link
Contributor

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

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