https://bugs.documentfoundation.org/show_bug.cgi?id=92965
Bug ID: 92965
Summary: use awt::XGradient instead of gradient name in fill
properties
Product: LibreOffice
Version: unspecified
Hardware: Other
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: medium
Component: Chart
Assignee: [email protected]
Reporter: [email protected]
Currently chart2 stores the gradient name and you have to look the gradient up
in a table when you want to use it.
This is crazy API design as it adds a level of indirection that makes it
difficult to resolve for the normal user. We should instead store the
css::awt::XGradient directly in the FillProperties class.
This would be trivial if we would not need to care about keeping the old string
based property in sync. So now we need to add the new property and write some
code that makes sure that the two stay in sync.
For that we need to make the setPropertyValue method explicit in all classes
that use FillProperties and handle when either the old or the new property are
set to also set the other one.
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
