Error "Cannot set properties of null (setting 'onclick')"

Question

Error "Cannot set properties of null (setting 'onclick')"

edersong opened this issue 9 months ago · comments

After the upgrade to v0.7.4, one of my cards are reporting the error Cannot set properties of null (setting 'onclick') even there is no onclick on his code.

MaximeNagel · Answer 1 · Wed Nov 08 2023 18:29:47 GMT+0800 (China Standard Time)

I have the same problem if I use name: '' in my card, maybe it's the reason for this.

Example:

columns:
- data: entity_picture
align: center
name: ''
icon: mdi:gas-station
modify: '''<img src="'' + x + ''"style="height: 35px;width:100%">'''

I add a space for the name and it works (I want only an icon for my colomn)

columns:
- data: entity_picture
align: center
name: ' '
icon: mdi:gas-station
modify: '''<img src="'' + x + ''"style="height: 35px;width:100%">'''

JeffCrum1 · Answer 2 · Wed Nov 08 2023 21:24:26 GMT+0800 (China Standard Time)

I had HTML TITLE tags on my names to give info on a mouse over.

Once I removed those, the error went away.

Ed Leckert · Answer 3 · Thu Nov 09 2023 22:31:19 GMT+0800 (China Standard Time)

The fix for this is in my repo. If you need it sooner or want to help me test it and are in a position to update flex-table-card.js manually, it is here:

https://github.com/EdLeckert/flex-table-card/tree/fix-generate-new-id-for-onclick-error

MaximeNagel · Answer 4 · Mon Nov 13 2023 17:53:16 GMT+0800 (China Standard Time)

The fix for this is in my repo. If you need it sooner or want to help me test it and are in a position to update flex-table-card.js manually, it is here:

https://github.com/EdLeckert/flex-table-card/tree/fix-generate-new-id-for-onclick-error

It works for me ! Thx for this quick hotfix 😃

JeffCrum1 · Answer 5 · Tue Nov 14 2023 05:34:41 GMT+0800 (China Standard Time)

Did not work in my situation.

Here is the column code that is at issue:

        - name: >-
            <div title="Clinch Indicator:&#10; * : Clinched Best League
            Record&#10; z : Clinched Playoff Birth&#10; x : Clinched
            Division&#10; - : In The Hunt&#10; e : Eliminated">C</div>
          data: entries
          modify: >-
            if(typeof x.stats.find(y=>y.abbreviation == 'CLINCH') !==
            'undefined' ){x.stats.find(y=>y.abbreviation ==
            'CLINCH').displayValue}else{'-'}

If I remove the HTML, it works fine.

Let me know if you need more code for the whole card.

Thanks

Ed Leckert · Answer 6 · Wed Nov 15 2023 00:53:10 GMT+0800 (China Standard Time)

@JeffCrum1: What behavior or error are you seeing?

I'm getting the character "C" with the key for "Clinch Indicator" on hover. I don't have the data to make the modify work, but the name works fine for me.

JeffCrum1 · Answer 7 · Wed Nov 15 2023 01:55:27 GMT+0800 (China Standard Time)

Understood Ed. Thanks. I will get you more info later tonight. I can probably even mock up something to make it work/not work.

…

On Tue, Nov 14, 2023 at 10:53 AM Ed Leckert ***@***.***> wrote: @JeffCrum1 <https://github.com/JeffCrum1>: What behavior or error are you seeing? I'm getting the character "C" with the key for "Clinch Indicator" on hover. I don't have the data to make the modify work, but the name works fine for me. — Reply to this email directly, view it on GitHub <#115 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AWTFU5D7VV7H3E5DAZHOC4LYEOOYFAVCNFSM6AAAAAA7CQDMUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJQGY4DINBRGQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

JeffCrum1 · Answer 8 · Fri Nov 24 2023 03:21:11 GMT+0800 (China Standard Time)

So sorry. I didn't realize I needed to delete the .gz file when manually updating.

All good now!

Thanks for your very quick response!!!

Markus Meissner · Answer 9 · Sat Nov 25 2023 00:53:54 GMT+0800 (China Standard Time)

there also is a 0.7.5 release now with all the fixes included...