[Bug 61684] New: CellStyle exposes colors as short and Color instances

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 61684] New: CellStyle exposes colors as short and Color instances

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61684

            Bug ID: 61684
           Summary: CellStyle exposes colors as short and Color instances
           Product: POI
           Version: 3.17-dev
          Hardware: PC
                OS: Mac OS X 10.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: SL Common
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

short getFillBackgroundColor();
    Color getFillBackgroundColorColor();

Do we want to continue to support both?
With BorderStyle and FillPatternType, we no longer support short values.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61684] CellStyle exposes colors as short and Color instances

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61684

--- Comment #1 from Javen O'Neal <[hidden email]> ---
Does short refer to an index into one of the tables defined in styles.xml? Can
short be arbitrarily large (within file format limits) if there are a large
number of styles, or is it a fixed quantity (only works for predefined
quantities), which could be captured by a color enum or static class constant)?

If so, this would be different BorderStyle and FillPatternType enums where the
number is irrelevant (either a POI implementation detail or a format
implementation detail).

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61684] CellStyle exposes colors as short and Color instances

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61684

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]