cellStyle with text-decoration isn't parsed correctly and breaks

[adamwinn]

I'm submitting a ... (check one with "x")

[x] bug report => see 'Providing a Reproducible Scenario'
[] feature request => do not use Github for feature requests, see 'Customers of ag-Grid'
[] support request => see 'Requesting Community Support'

Customers of ag-Grid

If you are a customer you are entitled to use the ag-Grid's customer support system (powered by Zendesk). Please use that channel for guaranteed response from the ag-Grid team with regards bugs, feature requests and support.

Requesting Community Support

If you are not a customer of ag-Grid, ag-grid staff will label your issue as managed-by-the-community. This means that ag-Grid staff is not going to be actively looking into it and it will get closed if inactive for more than one month. The community is welcome to help with this question/support issue.

Providing a Reproducible Scenario
Accepted reproducible scenarios are

• A description of the detailed steps to reproduce your behaviour in one of our examples in the docs.
• A plunker

If you decide to send us a plunkr, from any example in our website use the plunkr button in there to fork your own code by following the steps below:

• Select the framework that is appropriate to you from the drop-down
• Open it in plunker. (Use the button plunker in our example)
• Add your changes so that the behaviour is reproduced
• Save and Freeze the plunker(On the top left corner)
• Send us the link to the plunker(You can copy the URL from the browser)

If reporting a bug make sure to state.

Current behaviour.
Expected behaviour. If possible back this up with our docs/examples if possible

Current behavior
cellStyle with text-decoration isn't parsed correctly and breaks

cellStyle: function(params) {
                if (!params.data.line_code) {
                    return {color: '#337ab7', text-decoration: 'underline'};
                } else {
                    return null;
                }
            },

SyntaxError: missing : after property id[Learn More]

Picture of error:
picture of source in error console

Taking out the hyphen from text-decoration, just to see if it compiles, works with no errors.

Expected behavior
To see underlined text

Please tell us about your environment:

• ag-Grid version: v20.1.0

• Browser:
  Firefox

• Language: JS

[makinggoodsoftware]

Hi,

Can you provide a reproducible scenario using one of our examples or your source code in a plunker and we will investigate?

Accepted reproducible scenarios are

• A description of the detailed steps to reproduce your behaviour in one of our examples in the docs.
• A plunker

If you decide to send us a plunkr, from any example in our website use the plunkr button in there to fork your own code by following the steps below:

• Select the framework that is appropriate to you from the drop-down
• Open it in plunker. (Use the button plunker in our example)
• Add your changes so that the behaviour is reproduced
• Save and Freeze the plunker(On the top left corner)
• Send us the link to the plunker(You can copy the URL from the browser)

If reporting a bug make sure to state.

Actual behaviour.
Expected behaviour. If possible back this up with our docs/examples if possible

---

Note that this issue is getting closed, but you are welcomed to create a new issue if you can provide us with a reproducible scenario

If you are an ag-Grid Enterprise Customer, please raise this in our Zendesk support system - contact info@ag-grid.com for access. This is our primary channel for Support.

[adamwinn]

I figured it out. text-decoration needs to be in quotes. return {color: '#337ab7', 'text-decoration': 'underline'}