Skip to content

Additional stories #248

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

Draft
wants to merge 29 commits into
base: main
Choose a base branch
from
Draft

Additional stories #248

wants to merge 29 commits into from

Conversation

ChrisBaidoo
Copy link
Contributor

@ChrisBaidoo ChrisBaidoo commented Jul 13, 2022

Description

Most of the stories that are here are not rolled out across the Repos yet with the exception of the Button and the Dialogue components that have been partially rolled out across some of the Repos.
The Dialogue has been rolled out in MANAGE and CURATE
The Button has been rolled out of in MANAGE and CURATE

  • Text Field (Done but not rolled out yet.)
  • Autocomplete(Partially done and not rolled out yet)
  • Tooltips(Done but not rolled out yet)
  • Table(Done but not rolled out yet)
  • Switch
  • Snackbar
  • Slider
  • Select
  • Linear Progress
  • Chip(Partially done)
  • Checkbox
  • Avatar
  • Alert
  • Accordion

Somethings that was discussed but I never got round to doing.

Add the variant and colour controls for the Button story.

Checklist:

Put an x in the boxes that apply to this pull request (you can also fill these out after opening the pull request). If you're unsure about any of these, don't hesitate to leave a comment on this pull request!

  • I have read the gliff.ai Contribution Guide.
  • I have requested to pull a branch and not from main.
  • I have checked all commit message styles match the requested structure.
  • My code follows the style guidelines of this project.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have performed a self-review of my own code.
  • I have assigned 3 or less reviewers.
  • New and existing unit tests pass locally with my changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • My changes generate no new warnings.
  • I have made corresponding changes to the documentation.
  • New database changes have been committed.
  • If appropriate, I have bumped any version numbers.

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 1 alert when merging 6c47ed0 into cf20598 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-island-0a3514503-248.westeurope.azurestaticapps.net

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-island-0a3514503-248.westeurope.azurestaticapps.net

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 1 alert when merging 296a179 into cf20598 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-island-0a3514503-248.westeurope.azurestaticapps.net

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request introduces 1 alert when merging da261db into cf20598 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-island-0a3514503-248.westeurope.azurestaticapps.net

@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2022

This pull request introduces 1 alert when merging 7078bcf into cf20598 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-moss-0cf388303-248.westeurope.1.azurestaticapps.net

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-island-0a3514503-248.westeurope.azurestaticapps.net

@lgtm-com
Copy link

lgtm-com bot commented Jul 18, 2022

This pull request introduces 1 alert when merging 319677a into cf20598 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-moss-0cf388303-248.westeurope.1.azurestaticapps.net

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.

1 participant