Right now, if you translate a View with this style plugin, the parts of the chart that should have translation, such as the chart title and x- and y-axis labels are not translatable.

Comments

andileco created an issue. See original summary.

andileco’s picture

Assigned: Unassigned » diaodiallo

@diaodiallo, assigning to you.

diaodiallo’s picture

@andileco maybe only these are necessary?

diaodiallo’s picture

Status: Active » Needs review
diaodiallo’s picture

Hi @andileco, any thought about this code? want to close this issue.

andileco’s picture

@diaodiallo, there is not a display plugin in the source code, so I think that can be removed from the schema.

diaodiallo’s picture

Is this code enough and clean? @andileco

Thanks.

andileco’s picture

Status: Needs review » Needs work

@diaodiallo, you need to include the schema for fields_series_field, fields_drilldown_series_field, fields_data_field, and fields_operator. I don't think you need the charts_settings, as that should be inherited from the parent style.

diaodiallo’s picture

Version: 1.0.0-alpha2 » 1.0.0-alpha6
Status: Needs work » Needs review
StatusFileSize
new1.58 KB

This schema should be enough for the module.

andileco’s picture

Status: Needs review » Reviewed & tested by the community

@diaodiallo - looks great now!

  • e8bac81 committed on 1.0.x
    Issue #3257300 by diaodiallo: Add schema so that view can be properly...
diaodiallo’s picture

Status: Reviewed & tested by the community » Fixed
diaodiallo’s picture

Status: Fixed » Closed (fixed)