ICEfaces
  1. ICEfaces
  2. ICE-9029

ace:chart - implement varyBarColor property

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.2.0.GA
    • Fix Version/s: EE-3.3.0.GA_P02, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      *
    • Assignee Priority:
      P2

      Description

      The varyBarColor property of CartesianSeries could be exposed to users so that they might have bars of differing colours within the same series.

        Activity

        Hide
        Judy Guglielmin added a comment - - edited

        including ability to define seriesColors. Appears that charts can have centrally defined object for seriesColors or can use the model to define by chart model instance.

        Creating tests for all variations for barChart, but with implementation, since Line chart and Bar chart share CartesianSeries model, must create testing for that as well. Since seriesColor is something that can be defined at higher level, may also have to check all other models (so extra testing).

        First implementation of varyColor uses default jqPlot colors, 2nd uses model definition (by chart). Should be able to define by chart, so stackedSeries should be able to have defined color for each chart data series defined.

        Show
        Judy Guglielmin added a comment - - edited including ability to define seriesColors. Appears that charts can have centrally defined object for seriesColors or can use the model to define by chart model instance. Creating tests for all variations for barChart, but with implementation, since Line chart and Bar chart share CartesianSeries model, must create testing for that as well. Since seriesColor is something that can be defined at higher level, may also have to check all other models (so extra testing). First implementation of varyColor uses default jqPlot colors, 2nd uses model definition (by chart). Should be able to define by chart, so stackedSeries should be able to have defined color for each chart data series defined.
        Hide
        Judy Guglielmin added a comment -

        mistakenly committed to 3.3.0 mtce branch as ICE-9209

        Show
        Judy Guglielmin added a comment - mistakenly committed to 3.3.0 mtce branch as ICE-9209
        Hide
        Judy Guglielmin added a comment -

        rev 40580 for bulk of checkin for 3.3.0-mtce branch . 40518 to I8N the panel header
        rev 40516 for icefaces4 checkin.

        rev 37790 for 3.3 QA branch chart test app. Don't bother scripting until other bar chart new tasks are implemented.

        varyBarColor is a boolean in the model (CartesianSeries). seriesColor is a String[] in the ChartSeries-which CartesianSeries inherits from, so custom colors may be listed there.

        Also added defaultSeriesColors in chart object which can be used for any chart object to override the jqplot seriesColors.

        Show
        Judy Guglielmin added a comment - rev 40580 for bulk of checkin for 3.3.0-mtce branch . 40518 to I8N the panel header rev 40516 for icefaces4 checkin. rev 37790 for 3.3 QA branch chart test app. Don't bother scripting until other bar chart new tasks are implemented. varyBarColor is a boolean in the model (CartesianSeries). seriesColor is a String[] in the ChartSeries-which CartesianSeries inherits from, so custom colors may be listed there. Also added defaultSeriesColors in chart object which can be used for any chart object to override the jqplot seriesColors.
        Hide
        Carmen Cristurean added a comment - - edited

        Re-opening because of the showcase > chart > Bar demo: test is functional, however the drop-down on this demo appears to render source code, see screen shot (IF4 trunk only, rev# 40799) .

        Show
        Carmen Cristurean added a comment - - edited Re-opening because of the showcase > chart > Bar demo: test is functional, however the drop-down on this demo appears to render source code, see screen shot (IF4 trunk only, rev# 40799) .
        Hide
        Judy Guglielmin added a comment -

        rev 40814 missed messages.properties showcase example on previous checkin ->there now..

        Show
        Judy Guglielmin added a comment - rev 40814 missed messages.properties showcase example on previous checkin ->there now..
        Hide
        Carmen Cristurean added a comment -

        Verified with IF4 trunk # 40816 in FF28, Chrome34, IE10.

        Show
        Carmen Cristurean added a comment - Verified with IF4 trunk # 40816 in FF28, Chrome34, IE10.

          People

          • Assignee:
            Judy Guglielmin
            Reporter:
            Nils Lundquist
          • Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: