Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#232 Spider WebFilled - For given series instead of all

open
David Gilbert
None
5
2008-12-18
2008-12-18
psa
No

This patch fixed the a part of the bug [ 2392215 ] Foregroundalpha webfilled. For the spider web chart the Webfilled setting can now display some spider filled and other not filled.

Using the same logic as setSeriesPaint(0, Color.green) the setWebFilled(0) sets the first series to be filled.
By default if this method is not called all webs will be filled.

The first part of the bug cannot be replicated. I am able to set the foregroundalpha with the setter available.

Discussion

  • psa
    psa
    2008-12-18

    DiffFile

     
    Attachments
  • psa
    psa
    2008-12-18

    File Added: after webfilled.jpg

     
  • psa
    psa
    2008-12-18

    Screenshot with only selected web filled

     
    Attachments
  • David Gilbert
    David Gilbert
    2008-12-18

    • assigned_to: nobody --> mungady
     
  • David Gilbert
    David Gilbert
    2008-12-18

    Is there any chance to get a cleaner diff file? It looks like formatting changes have just marked everything in the file as a change, so it is hard for me to tell what you've changed. Also, I just happened to spot some generics in your sources (for example, 'List<Integer>') which we have to avoid as JFreeChart still requires just Java 1.3.1 or later.

     
  • psa
    psa
    2008-12-18

    Since it was 1.3 and later I thought generics were ok.But i will remove them and send a diff of the modified code soon.The diff attached was generated from Eclipse.I'll check if I can find a cleaner one.

     
  • David Gilbert
    David Gilbert
    2008-12-19

    No, generics came in at version 1.5. It would be nice to be able to use them, but there's still a significant number of users on 1.4, so we'll be patient!

    Regards,

    Dave

     
  • psa
    psa
    2008-12-27

    Cleaner Diff File

     
    Attachments
  • psa
    psa
    2008-12-27

    Hi
    I dint mean to create a new patch file.Sorry about it and please remove it.It has been added to this thread.

    I have removed generics and have attached a cleaner diff file which only
    shows portions of code that have been modified/added.I hope this helps.

    Thanks
    File Added: SpiderWebFillDiff.txt