refactor: migrate 'toggleDatasetEntry' calls to 'toggleAttribute'

[TomMenga]

Removed toggleDatasetEntry usages in favor of element.toggleAttribute

[jeripeierSBB]

You won! :-) Thanks! Not sure if we should keep the migration script?

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 95.55556% with 4 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (main@af86d2f). Click here to learn what that means.

Files	Patch %	Lines
src/components/file-selector/file-selector.ts	0.00%	4 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2488   +/-   ##
=======================================
  Coverage        ?   93.67%           
=======================================
  Files           ?      298           
  Lines           ?    24334           
  Branches        ?     2027           
=======================================
  Hits            ?    22795           
  Misses          ?     1506           
  Partials        ?       33           

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

[TomMenga]

You won! :-) Thanks! Not sure if we should keep the migration script?

The idea was to keep it for one/two weeks top just in case someone needs it.
Most likely, nobody will use it though.
Also, it might be useful to have it in the history for future reference (as an example of when (if) we'll implement migration scripts for breaking changes)

[jeripeierSBB]

You won! :-) Thanks! Not sure if we should keep the migration script?

The idea was to keep it for one/two weeks top just in case someone needs it. Most likely, nobody will use it though. Also, it might be useful to have it in the history for future reference (as an example of when (if) we'll implement migration scripts for breaking changes)

it's ok :)