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

[Ag Grid Theme] Ag grid v30 - no sass #2661

Merged
merged 185 commits into from
Nov 24, 2023
Merged

[Ag Grid Theme] Ag grid v30 - no sass #2661

merged 185 commits into from
Nov 24, 2023

Conversation

alycrys
Copy link
Contributor

@alycrys alycrys commented Oct 31, 2023

Closes #2602

Storybook Preview: #2661 (comment)

content: "\e922";
}

/* [class*="ag-theme-uitk"] .ag-icon-filter-clear:before {
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes. Also waiting for design to provide a grab icon (for the drag & drop rows)

@@ -60,7 +60,7 @@ export function useAgGridHelpers({
}
}, [density, agThemeName, compact]);

const className = `${agThemeName}-${density}${
const className = `${agThemeName}${
Copy link
Contributor

Choose a reason for hiding this comment

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

Worth calling out in noticeable places

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure I understand

@joshwooding
Copy link
Contributor

The vercel build fails with:

image

@alycrys
Copy link
Contributor Author

alycrys commented Nov 24, 2023

The vercel build fails with:

image

sorted. thanks

@alycrys alycrys merged commit 883967a into ag-grid-next Nov 24, 2023
@alycrys alycrys deleted the ag-grid-30-css branch November 24, 2023 14:36
origami-z pushed a commit that referenced this pull request Apr 3, 2024
origami-z pushed a commit that referenced this pull request Apr 4, 2024
origami-z pushed a commit that referenced this pull request Apr 11, 2024
origami-z pushed a commit that referenced this pull request Apr 25, 2024
origami-z pushed a commit that referenced this pull request Apr 29, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request Apr 29, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 3, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 9, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 13, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 17, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 22, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 23, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 23, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 23, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
origami-z pushed a commit that referenced this pull request May 24, 2024
Ag grid theme - add cypress tests (#2614)

[Ag Grid Theme] test UITK theme (#2699)

[Ag Grid Theme] Ag grid v30 - no sass (#2661)

Co-authored-by: Josh Wooding <[email protected]>

use content characteristic

Create tasty-trains-cover.md

Update tasty-trains-cover.md

fixes

AG Grid Theme site docs (#2796)

Update build.mjs

Update package.json

Ag grid test range selection (#2827)

Ag grid theme tooltip test (#2654)

[AG Grid Theme] Update icons (#2849)

fix editable cell bug

Fixed missing padding and focus ring on focused editable cell.
Fixed wrong background and missing corner flag when editable cell loses focus.

Update salt-ag-grid-theme.css

Update salt-ag-grid-theme.css
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.

3 participants