-
Notifications
You must be signed in to change notification settings - Fork 30
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
fix(DataTable): fix HTML semantics #2989
Draft
dreamwasp
wants to merge
31
commits into
main
Choose a base branch
from
cass-gm-945
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
+21,777
−33,634
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* start with empty lib * add babel config? * add old package.json back * add old babel config * leave to show jake * empty storybook, maybe not running * just try with base config * start adding maybe need packages * try one story * progress not perfection * ci: set correct node version everywhere * chore: separate dev dependencies from production dependencies * remove webpack here * remove comma * remove comma, yarn * mess but uhhhh starting * RUNNING * move logo back and keep testing * experiment, maybe getting closer * remove storybook/addon-styling-webpack * jaek changes + temp stories * jake changes + fixes * start tweaking GHA * fix remaining type errors * test CI * update netlify cli * try removing msg * test * test * try fr * test w/o lerna * remove msg * add alias path for variance * spellcheck * try slightly different syntax * test CI (#2960) * test CI * filter styleguide site --------- Co-authored-by: Jake Hiller <[email protected]> * remove unused pkgs; * test deploy message * fix msg syntax * remove comment * remove explicit webpack bump * tweak pkg .json + failing tests * fix provider test * test new variance pkgs * rm other stuff * linter error * fix baseUrl * resolve pathing error * Cass sb 8 fix connectformtypes (#2963) * build errors in forms * fixed build errors * resolve build error * keep getError message internal * more test tweaks * no gamut-test tests for now * skip selectdropdown tests for now * await video resolution * skip video for now * skip last problematic test * temporarily move offending stories (#2956) * temporarily move offending stories * temp old storybook * radial * migrate layout components (#2961) * move old layout components for ref * update pkgs + try dark theme * basic light mode working * color modes working in components * configure toolbar + start adding params * dark mode theme config working, start on DocsPage configs * keep migrating * anchors working * aboutheader working, need to start working on other headers + types * fix type errors in .storybook layout components * test stories, need to fix prettier * fix formatting * fixed theming ! * better heading * fix other pkg weirdness * fix default homepage + docs ordering * add absolute paths * add absolute paths * create story template (#2964) * create story template * move template out of atoms * update import * withSource to sourceState --------- Co-authored-by: cassie spain <[email protected]> * add source to template * fix Canvas in template * allow static images in storybook 8 (#2973) * test image stuff * getting closer * test dist * remove asset tests * test in pr * tweak some more examples * create foundations + layout stories (#2966) * create foundations + layout stories * feat(Molecules Pages) (#2971) added existing Molecules pages * typography + meta (#2975) * create foundations + layout stories * start foundations * start retooling componentheader * component header tweak * start restructure of header components * TOC + Theme working, need DataTable + code background color * working on color mode - need to fix text color * revert test * Theme page sorted, need to fix ColorMode * colormode sorted * colormode sorted * start type fix + what to do about blocks * LayoutGrid + content Container * LayoutGrid done * start flexbox, need to look into sandbox stuff * gridbox + flexbox are good, just need to tweak * tweaks to headers * lots of style tweaks + deduplications * fix typograpth, need to fix color mode switches * start on system stories * unify shared elements * add variance + fix tables * more tweaks to status * design tweaks * add hub * more tweaks * rename DataTable * clean up of default * typography + meta stories * add anchor story * start text + hiddentext * text done * started snippet changes * fixed code snippets * meta and sorted done?" * renamed + sorting * more kenny edits * more kenny edits + other new tab for links * the last of it * fix molecule order * update production deploy (#2980) * [gm-916] update storybook production deploy * swap to relative pathing * again * feat(Organisms) (#2978) * added in Organisms stories * remove old styleguide files * Revert "remove old styleguide files" This reverts commit baba497. * feat(Atoms): migrate all Atom stories (#2972) * create story template * move template out of atoms * update import * skip to content * withSource to sourceState * update skip to content * tag story * add statuses * radial progress story * updates from figma * toggle story * update status * pattern story * update toggle interactive example * illustrations story * focus trap story * update playground examples * floating card story * update illustrations subtitle * drawer story * update drawer story * animation story * add figma links * card story * about page * progress bar story * popovercontainer story * loader stories * fix about pages * move examples below * icon stories * checkbox story * add checkbox example * most of input story * Badges started (#2979) * Badges started * badges done * add most GH links, need to test * add github links to most pages * remove comment * finish input stories * update more github links * textarea story * radio story * update examples * select story * fix card usage guidelines * feat(Atoms): adding in the remaining Atoms (#2982) * finished badges * added buttons * added formelements * updated about page * minor touchups * fixed minor ts error * updated usage for CTAButton and IconButton * selectdropdown story * remove optional text * update forminputs table of contents * fix forminput table of contents links * remove logo story * remove placeholder text * update main table of contents * fix icon table of contents * fix loaders table of contents links * PR feedback * uncomment link --------- Co-authored-by: cassie spain <[email protected]> Co-authored-by: Kenny Lin <[email protected]> * remove old styleguide files (#2985) --------- Co-authored-by: Amy Resnik <[email protected]> Co-authored-by: Kenny Lin <[email protected]> --------- Co-authored-by: Amy Resnik <[email protected]> Co-authored-by: Kenny Lin <[email protected]> --------- Co-authored-by: Jake Hiller <[email protected]> Co-authored-by: Amy Resnik <[email protected]> Co-authored-by: Kenny Lin <[email protected]>
* fix reges for the AI * copilot suggestion * update rhc for submit issues * Revert "update rhc for submit issues" This reverts commit 97f0529. * start fixing small lint things * less errors * format TOC * disable useState es-lint for example * fixed formstate
☁️ Nx Cloud ReportWe didn't find any information for the current pull request with the commit d14d96b. Check the Nx Cloud Source Control Integration documentation for more information. Alternatively, you can contact us at [email protected]. Sent with 💌 from NxCloud. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
PR Checklist
Testing Instructions
Don't make me tap the sign.
PR Links and Envs