-
Notifications
You must be signed in to change notification settings - Fork 43
FLUT-965284 - [Feature] Added 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
base: development
Are you sure you want to change the base?
Conversation
Your pull request has been created. Syncfusion support team will validate this commit and publish your changes accordingly. |
There was a problem hiding this 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.
Also add the output images in the PR Template for the newly included APIs in user guide. |
Your pull request has been created. Syncfusion support team will validate this commit and publish your changes accordingly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Image is stretched, check all images which you have added newly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Address review concerns
Feature description
Review and update the API documentation for the slider, range slider and range selector widgets.
Key objectives:
Analysis and design
Solution description
Output screenshots
edgeLabelPlacement
Range slider:


Slider:


Range Selector:

Areas affected and ensured
Code snippets and content have been affected due to the modification.
API Changes
Missed API in this documentation changes
edgeLabelPlacement
- AddedoverlayShape
- Not Needed, We can directly customize from slider theme dataBehavioral 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