Salt AG-Grid theme supports v29+ ag grid

[origami-z]

Currently ag grid is using Legacy SASS method to generate CSS, and need to move to new way.

Also new versions of ag grid introduced proper CSS like below

[class*=ag-theme-] {
   ...
    --ag-background-color: #fff;
   ...
}

These will override Salt colors applied, so will need to follow new way of overriding colors.

StackBlitz Demo of ag grid v30 CSS variable breaking styling.

Tasks

Beta Give feedback

1. Verify all relevant visual regression snapshots available on Chromatic #2557
   package: ag-grid-theme +1
2. Consolidate existing class attribute matching and mixin approach
3. Upgrade to latest version of ag grid and use new theming approach #2602
   +0
   
4. Snapshot release and test
5. ag grid theme package v2 feedback and alpha release #2531
   package: ag-grid-theme +1
   
6. Update docs #2683
   package: ag-grid-theme role: content 📝 role: dev 💻 +3
   
7. Add grab icon in AG-Grid Theme fonts #2792
   role: design 🎨 role: dev 💻 +2
   
8. Add test for Improve Ag Grid Theme Grid Header #3127
9. Menu list style different between filter panel and column panel
10. Regression of wrapText and autoHeight do not set the correct heights #3084
11. Regression of Ag grid theme header text color should be secondary color #3065
12. Possible regression of Fix editable cell in ag-grid-theme #2906
13. [v31] Side panel incorrect fixed width. Possible fix: .ag-tool-panel-wrapper > .ag-react-container to inherit
14. [v31] cell range selection color incorrect
15. [v31] floating filters incorrect additional half border on the right
16. [v31] status bar misaligned verticially
17. Make it possible to import density / mode specific css entry Not needed given bundle size no longer a concern
18. Minimize production css bundle
19. Update default style in package.json to salt one

[joshwooding]

You beat me to creating this, we've already discussed the next set of actions, and moving to the new CSS approach is part of it

[origami-z]

Updated release date, as work has not started.

[origami-z]

update target date to match subtask

[origami-z]

25 Oct - Chromatic snapshot to be design reviewed with @bhoppers2008
23 Oct - cypress snapshot test merged to next branch. Fixing conflict from main, will continue feedback afterwards.

[joshwooding]

Moved the target date to 31st Oct to accommodate feedback from consumers

[joshwooding]

Hard-coded colours need to be removed and mapped to Salt colours.

[origami-z]

3 Nov - PR ready for review, moving to feedback stage

[shey-v]

6 Nov - PR reviews and waiting for more to be able to merge. @joshwooding

[joshwooding]

Almost ready, creating tests for UITK theme. Consumers not able to test the theme yet as they haven't begun their upgrade. PR still needs a review

[origami-z]

13 Nov - PR review requested this morning, Chromaticc tests added for uItk theme

[origami-z]

17 Nov - Fixing UITK theme to match on Chromatic / Cypress.

[origami-z]

20 Nov - wip, more test to be fixed. ETA tmr

[origami-z]

22 Nov - dev done. verifying test passing, review for dev & design review (both Salt and UITK theme).

[origami-z]

24 Nov - snapshot tests pass. Range selection bug raised. Can't merge to main, bug fixes needed for v1.

[origami-z]

27 Nov - doc changes on ag grid next branch.

[origami-z]

1 Dec - trying at snapshot (help from @lukeac123), at feedback stage. Keep in next branch.

[origami-z]

4 Dec - trying snapshot by @lukeac123. More thorough test needed.

[origami-z]

6 Dec - more tests to be added for range selection. discuss how to manage 2 ag grid theme version.

[origami-z]

8 Dec - Need additional icon (drag and drop row, GH issue? @pseys did it). Version management to be discussed.

[origami-z]

11 Dec - icons created, needs to be added to the icon font (how? to be confirmed). Version management to be discussed.

[joshwooding]

18 Dec - Icons have been merged, and the data grid theme has been updated with the new icons.

[origami-z]

Closed by #3195