Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LINK-1765, LINK-1766 | Refactor tables #336

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

jorilindell
Copy link
Contributor

@jorilindell jorilindell commented Feb 15, 2024

Description

Replace all tables to use (excluding tables with collapsible content) HDS Table component.
HDS Table doesn't support clickable rows so replace them by making item name or id to work as a link.
Add column header for the tables with action buttons. Make that column to be visible only to screen readers.

Closes

LINK-1765
LINK-1766

@terovirtanen
Copy link
Contributor

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr336-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is success for https://linkedcomponents-pr336-ui.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr336-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is success for https://linkedcomponents-pr336-ui.dev.hel.ninja 😆🎉🎉🎉

@jorilindell jorilindell marked this pull request as ready for review February 15, 2024 19:53
@jorilindell jorilindell requested review from harriris-vincit and a team February 15, 2024 19:53
@jorilindell jorilindell changed the title fix: refactor all tables to use HDS Table component as base LINK-1765, LINK-1766 | Refactor tables Feb 15, 2024
@terovirtanen
Copy link
Contributor

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr336-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is success for https://linkedcomponents-pr336-ui.dev.hel.ninja 😆🎉🎉🎉

@jorilindell jorilindell force-pushed the fix/LINK-1765_refactor-tables branch from ab088d8 to 11e34c1 Compare March 1, 2024 14:06
@jorilindell jorilindell force-pushed the fix/LINK-1765_refactor-tables branch from 11e34c1 to 2003c73 Compare March 1, 2024 14:39
Copy link

sonarcloud bot commented Mar 1, 2024

@terovirtanen
Copy link
Contributor

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr336-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is failed for https://linkedcomponents-pr336-ui.dev.hel.ninja 😿💢💥💥

1 similar comment
@terovirtanen
Copy link
Contributor

e2e tests result is failed for https://linkedcomponents-pr336-ui.dev.hel.ninja 😿💢💥💥

Copy link

@harriris-vincit harriris-vincit left a comment

Choose a reason for hiding this comment

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

A test seems to be failing in the review build and the reason for the failure should be checked/fixed, but otherwise looks good now.

@terovirtanen
Copy link
Contributor

e2e tests result is success for https://linkedcomponents-pr336-ui.dev.hel.ninja 😆🎉🎉🎉

@jorilindell jorilindell merged commit e20810d into main Mar 4, 2024
20 checks passed
@jorilindell jorilindell deleted the fix/LINK-1765_refactor-tables branch March 4, 2024 14:26
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.

None yet

3 participants