-
Notifications
You must be signed in to change notification settings - Fork 90
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
Conversation
content: "\e922"; | ||
} | ||
|
||
/* [class*="ag-theme-uitk"] .ag-icon-filter-clear:before { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing?
There was a problem hiding this comment.
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}${ |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure I understand
Co-authored-by: Josh Wooding <[email protected]>
Co-authored-by: Josh Wooding <[email protected]>
Co-authored-by: Josh Wooding <[email protected]>
Co-authored-by: Josh Wooding <[email protected]>
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
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
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
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
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
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
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
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
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
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
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
Closes #2602
Storybook Preview: #2661 (comment)