Skip to content

Conversation

@o-jorgensen
Copy link
Contributor

Resolves #3779

Disclaimer: this is my first fork & pull-request, please let me know if I should do things differently 😇

@pomfrida
Copy link
Collaborator

Thanks for the PR! 🙌
Adding a className to the pagination wrapper makes sense for custom styling or test selectors 👌

The lint errors are purely formatting-related (Prettier). You can fix them with:
pnpm lint:all --fix

Copy link
Collaborator

@pomfrida pomfrida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@o-jorgensen
Copy link
Contributor Author

@pomfrida Thank you for helping me out, do I do anything further with this PR myself? 🙂

@pomfrida pomfrida merged commit 6f7ec33 into equinor:develop Apr 25, 2025
6 checks passed
torleifhalseth pushed a commit that referenced this pull request Apr 29, 2025
* Add class to pagination wrapper

* Fix linting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add className to the pagination wrapper in EDS data grid

2 participants