search

NathanaelA / fluentreports

:page_facing_up: Fluent Reports - Data Driven Reporting Engine for Node.js and Browsers :page_facing_up:
https://fluentreports.com
MIT License
375 stars 84 forks source link

[FRG] Fix clearing font size field wiping out report #248

Closed beauwest closed 2 years ago

beauwest commented 3 years ago

I tried to clear the font size fields so that I could just set a default report font size, but it deleted most of my report as a result.

Frustrated-Programmer commented 3 years ago

May I ask what you did to "clear" a fontSize field? and which elements you cleared these fields from?

I haven't been able to reproduce this issue.

beauwest commented 3 years ago

Sure, my steps

If you inspect the groupBy[]->header node in the output in the console, the fontSize property ends up with NaN. On the demo, it's not that big of a deal, but in our case, we save the JSON structure to the database, and then use that JSON to restore the report structure for editing. The NaN causes problems.