search

apgapg / pie_chart

Flutter Pie chart with animation
https://pub.dev/packages/pie_chart
MIT License
179 stars 87 forks source link

I want to add a text style for the center text #46

Open mubaroqiqbal opened 4 years ago

mubaroqiqbal commented 4 years ago

i already fixed it. how can i push it so you can merge it?

apgapg commented 4 years ago

You need to create a pull request.

  1. Fork the repo
  2. Create a new branch from master branch with name like add-text-style-to-center-text (for eg)
  3. Commit your changes
  4. Create a PR on master branch
pagman commented 3 years ago

I want to do that too i want to add a text style or atleast to change the color of the center text . Can someone help me with that?

mubaroqiqbal commented 3 years ago

You need to create a pull request.

  1. Fork the repo
  2. Create a new branch from master branch with name like add-text-style-to-center-text (for eg)
  3. Commit your changes
  4. Create a PR on master branch

~/Documents/pie_chart> git -c credential.helper=/Applications/SmartGit.app/Contents/Resources/credentials.sh push --porcelain --progress --recurse-submodules=check origin refs/heads/center-textstyle:refs/heads/center-textstyle remote: Permission to apgapg/pie_chart.git denied to mubaroqiqbal. unable to access 'https://github.com/apgapg/pie_chart.git/': The requested URL returned error: 403

i can't push my change so i can't make a pull request to your repo, it said acces denied when push the change

apgapg commented 3 years ago

Okay i will add text style for center text.

@mubaroqiqbal you need to fork and create PR on master of piechart

mubaroqiqbal commented 3 years ago

@apgapg i already create the pull request, i'm sorry i didn't noticed that i'm working at your repo clone not in the fork

rivetingpeppermint commented 2 years ago

Is it possible to leave the center text without any background color (transparent)? I've tried setting the TextStyle but it turns out the center background color is already pre-set.

apgapg commented 2 years ago

@rivetingpeppermint I see backrgound used is defined by chartValueBackgroundColor parameter, and the same is utilised for chart values too. You can change this parameter to Colors.transparent for your case