- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.9k
Gauge Chart #1933
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: main
Are you sure you want to change the base?
Gauge Chart #1933
Conversation
| Can you please rebase the branch? | 
| It should be OK, I also removed files not concerned by this PR | 
| Codecov Report❌ Patch coverage is  
 Additional details and impacted files@@            Coverage Diff             @@
##             main    #1933      +/-   ##
==========================================
+ Coverage   92.44%   92.99%   +0.54%     
==========================================
  Files          50       54       +4     
  Lines        3681     3984     +303     
==========================================
+ Hits         3403     3705     +302     
- Misses        278      279       +1     
 Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
 | 
| Okay, now I found a little bit of time to take a look at this one. Here I have some concerns, please have a look: I tried to pass only one color to the  valueColor: VariableGaugeColor(
  colors: [
    AppColors.contentColorYellow,
  ],
  limits: [],
),Or if you want to keep the abstraction, please let me know what other types of  | 
| I also have a suggestion for the  Please let me know your thoughts | 

Updated code from #1424
The implementation of the gauge chart (discussed here #199)