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.

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

Feat: Multiple Pointer Feature Completed #82

Merged
merged 12 commits into from
Mar 10, 2023

Conversation

Afroz-Shaikh
Copy link
Collaborator

Multiple Pointer Feature

This PR closes #60

Changes

Added MultiPointer Feature:

Screenshot 2023-03-09 at 6 51 15 PM

Changed API to take a List of Pointers instead of a single Pointer

Screenshot 2023-03-09 at 6 54 35 PM

Deprecated ValueBarOffset as no longer required (Due to Value being Deprecated now)

Screenshot 2023-03-09 at 6 57 34 PM

Adjusted the Pointer Labels to better match their Position

Updated Methods in Pointer Class along some restructuring of the code

@Afroz-Shaikh Afroz-Shaikh requested a review from hasnentai March 9, 2023 13:51
@@ -15,36 +15,36 @@ class _MyValueBarPositionState extends State<MyValueBarPosition> {
return Scaffold(
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this example

@@ -35,7 +35,8 @@ class LinearGauge extends LeafRenderObjectWidget {
this.rangeLinearGauge = const [],
this.customLabels = const [],
this.valueBarPosition = ValueBarPosition.center,
this.valueBarOffset = 0.0,
@Deprecated('Use a list of ValueBars with individual offsets instead')
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not in production remove this

@hasnentai hasnentai added enhancement New feature or request Feature This is a new feature for the project and removed enhancement New feature or request labels Mar 10, 2023
@hasnentai hasnentai merged commit f09fa7b into GeekyAnts:main Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This is a new feature for the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MultiplePointers
2 participants