[CORL-3110] Add further static classnames across notifications components - Githubissues

coralproject / talk

A better commenting experience from Vox Media

https://coralproject.net

Other

1.88k stars 355 forks source link

[CORL-3110] Add further static classnames across notifications components #4559

Closed nick-funk closed 6 months ago

nick-funk commented 6 months ago

What does this PR do?

Adds static classnames across various notifications buttons, trays, feeds, and actions.

These changes will impact:

[X] commenters
[ ] moderators
[ ] admins
[X] developers

What changes to the GraphQL/Database Schema does this PR introduce?

None

Does this PR introduce any new environment variables or feature flags?

No

If any indexes were added, were they added to `INDEXES.md`?

N/A

How do I test this PR?

Inspect the modified notification icons, buttons, trays, tabs, etc and see they have static class names to bind external CSS to.

Were any tests migrated to React Testing Library?

No

How do we deploy this PR?

Merge into epic branch.

netlify[bot] commented 6 months ago

Deploy Preview for gallant-galileo-14878c canceled.

Name	Link
Latest commit	7b6dbae801c2c40b88373c0c990f730fa4c3dbe8
Latest deploy log	https://app.netlify.com/sites/gallant-galileo-14878c/deploys/65d916cfc847040009182610