pie_chart icon indicating copy to clipboard operation
pie_chart copied to clipboard

I want to add a text style for the center text

Open mubaroqiqbal opened this issue 4 years ago • 7 comments

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

mubaroqiqbal avatar Aug 13 '20 10:08 mubaroqiqbal

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

apgapg avatar Sep 05 '20 04:09 apgapg

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?

pagman avatar Nov 09 '20 20:11 pagman

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

mubaroqiqbal avatar Nov 10 '20 05:11 mubaroqiqbal

Okay i will add text style for center text.

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

apgapg avatar Nov 10 '20 05:11 apgapg

@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

mubaroqiqbal avatar Nov 10 '20 07:11 mubaroqiqbal

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.

rivetingpeppermint avatar Jan 13 '22 09:01 rivetingpeppermint

@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

apgapg avatar Jan 13 '22 10:01 apgapg