Remove explicit data attributes

[just-boris]

Description

Refactor to follow our convention to prefer using scoped class names instead of data-attributes

Related links, issue #, if available: n/a

How has this been tested?

• PR build
• See the preview deployment

Review checklist

The following items are to be evaluated by the author(s) and the reviewer(s).

Correctness

• Changes include appropriate documentation updates.
• Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
• Changes do not include unsupported browser features, see CONTRIBUTING.md.
• Changes were manually tested for accessibility, see accessibility guidelines.

Security

• If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

• Changes are covered with new/existing unit tests?
• Changes are covered with new/existing integration tests?

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (16d8029) 93.97% compared to head (c9cb899) 93.97%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #219   +/-   ##
=======================================
  Coverage   93.97%   93.97%           
=======================================
  Files          16       16           
  Lines         730      730           
  Branches      178      178           
=======================================
  Hits          686      686           
  Misses         38       38           
  Partials        6        6           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[pan-kot]

Looks great!