Split tablesort.module.css out to its own library and attach it via tablesort

Possible duplicate of 📌 Refactor system/base library Needs work , as some work for tablesort has already been done in that issue.

Good point - I think I opened this before I found that one. Closing.

Actually no. We should split the change out to here since it's tricky.

I've adjusted the tablesort.module.css but I could not figure where is the best place to attach the new library.

Someone please review and guide me here.

catch → changed the visibility of the branch 3432244-split-tablesort.module.css-out to hidden.

@xamount this is a tricky one, I ended up opening a new MR because there's been some related changes in core since your branch was open. With attachments, I wasn't sure either but added a template_preprocess implementation which works and is at least compact.

Should be reviewable/testable now.

So the way I tested this one was by just created a few nodes with A,B,C in front of the title. Then on the content view verified the table sorting worked as expected.

If different types of sorting are needed let me know, else believe this is good.

Paths needs to be updated after css file move

Updated the icon paths, good spot, not sure how I missed it in manual testing, maybe claro overrides hid the bug?

Yes I don't think any core theme uses this file

Feedback appears to be addressed.

Oops my mistake.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Rebased.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.