eslint-plugin-react icon indicating copy to clipboard operation
eslint-plugin-react copied to clipboard

[Bug]: autofix for jsx-closing-bracket-location deletes comments

Open Vladyslav-Soldatenko opened this issue 1 year ago • 1 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues and my issue is unique
  • [X] My issue appears in the command-line and not only in the text editor

Description Overview

When using jsx-closing-bracket-location with "after-props" setting, on autofix it deletes all comments which exist before actual non-comment prop (usually it's commented props).

For example, if run eslint --fix on this file image

it will remove all comments before src prop image

Same goes, of course, for html tags which have both opening and closing tags image

comments will be removed in this case as well

I often find myself commenting some props for testing, and this deleting bothers me. To avoid it, I need to move props here and there so that there's a non-commented line before closing bracket

Expected Behavior

I think the expected behavior should be to make last comment be commented with "/* .... */" and place closing bracket there as shown below. Technically it will be "after props", but will not remove comments which might be needed by developer

image

If maintainers find this issue suitable for fix, I'll try to fix it myself

eslint-plugin-react version

v7.35.0

eslint version

v8.24.0

node version

v16.17.1

Vladyslav-Soldatenko avatar Aug 02 '24 09:08 Vladyslav-Soldatenko

same problem here

eslint-plugin-react version v7.35.0

eslint version v8.57.0

node version v18.20.3

TotooriaHyperion avatar Aug 15 '24 04:08 TotooriaHyperion