Fix eslint react/jsx-no-bind

[atrovato]

Fixes 16 eslint warning: react/jsx-no-bind.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (396efc7) 98.19% compared to head (00cafb0) 98.19%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1992   +/-   ##
=======================================
  Coverage   98.19%   98.19%           
=======================================
  Files         786      786           
  Lines       12308    12308           
=======================================
  Hits        12086    12086           
  Misses        222      222           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[relativeci]

Job #2236: Bundle Size — 9.39MiB (+0.02%).

00cafb0(current) vs 396efc7 master#2235(baseline)

Warning

Bundle contains 3 duplicate packages – View duplicate packages

Bundle metrics   2 changes 1 regression

	Current Job #2236	Baseline Job #2235
Initial JS	5.18MiB(+0.03%)	5.18MiB
Initial CSS	302.44KiB	302.44KiB
Cache Invalidation	55.09%	53.88%
Chunks	51	51
Assets	158	158
Modules	1425	1425
Duplicate Modules	20	20
Duplicate Code	0.81%	0.81%
Packages	124	124
Duplicate Packages	3	3

Bundle size by type   1 change 1 regression

	Current Job #2236	Baseline Job #2235
JS	6.87MiB (+0.02%)	6.87MiB
IMG	2.08MiB	2.08MiB
CSS	318.94KiB	318.94KiB
Fonts	93.55KiB	93.55KiB
Other	16.14KiB	16.14KiB
HTML	13.58KiB	13.58KiB

---

View job #2236 report View atrovato:eslint-fix-jsx-no-bind branch activity

[Pierre-Gilles]

Are you sure it works? (tested in real life?)

If there is no bind, the function needs to be:

• A class attribute function
• Or the bind needs to be done in the constructor

[atrovato]

You're right.
I tested first, on demo mode (as I can't test with real devices), it was failing.
Now I set method as class attributes, it works.
Thanks @Pierre-Gilles

But I feel a bit sad that warnings are not checked on development phase...