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

feat(tools): knobs #2781

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

feat(tools): knobs #2781

wants to merge 64 commits into from

Conversation

bennypowers
Copy link
Member

@bennypowers bennypowers commented Jun 20, 2024

What I did

  1. add elements to tools
  2. add pft-element-knobs
  3. add knobs demos to dev server

Testing Instructions

  1. run the dev server
  2. http://localhost:8000/
  3. click on knobs

Notes to Reviewers

bennypowers and others added 26 commits March 31, 2024 12:04
* fix(avatar)!: remove BaseAvatar

Closes #2617
Closes #2575
Closes #2349

* fix(avatar): update css
* fix(clipboard-copy)!: remove BaseClipboardCopy

Closes #2611

* fix(clipboard-copy): use pfv4 styles

* fix(clipboard-copy)!: rename ClipboardCopyCopiedEvent to PfClipboardCopyCopiedEvent
* fix(switch)!: remove BaseSwitch

Closes #2618

* fix(switch): remove extra semicolon

* Merge branch 'staging/4.0' into fix/switch/remove-base

* fix(switch): svg namespace

---------

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
Closes #2613

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
* fix(label)!: remove BaseLabel

Closes #2620

* feat(label): self-remove on close

* test(label): update tests

* fix(label): correct font-size unit, include missing pf global var

* docs(label): fix accesibility demo

* fix(label): removes the removal of element and only triggers close event

* fix(label): undo last commit, see the example in the changeset

---------

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
Closes #2614

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
Closes #2615

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
* fix(card): moving styles and code from baseclass

* chore: changset

* feat(card): removing basecard class, styles

* fix(card): remove unused basecard import

* Update .changeset/sharp-spiders-float.md

Co-authored-by: Benny Powers <bennypowers@users.noreply.github.com>

* docs: make changeset major

* fix(card): make slot controller private

* feat(card): title slot, header actions

---------

Co-authored-by: Benny Powers <bennypowers@users.noreply.github.com>
Co-authored-by: Benny Powers <web@bennypowers.com>
* fix(icon)!: remove BaseIcon

Closes #2621

* docs: jsdoc

* fix(icon): base styles

* docs: addIconSet jsdoc

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>

* docs: jsdoc for getIconUrl

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>

* style: lint

* fix: remove baseicon from bad merge

---------

Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
wip dev server knobs
Copy link

changeset-bot bot commented Jun 20, 2024

⚠️ No Changeset found

Latest commit: 9e64b3a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added work in progress POC / Not ready for review functionality Functionality, typically pertaining to the JavaScript. tools Development and build tools labels Jun 20, 2024
@bennypowers bennypowers changed the base branch from main to staging/4.0 July 14, 2024 12:42
@github-actions github-actions bot removed doc styles An issue or PR pertaining only to CSS/Sass tests Related to testing generator Updates relating to the generator labels Jul 14, 2024
Copy link
Contributor

SSR Test Run for e181d72: Report

Base automatically changed from staging/4.0 to main August 16, 2024 12:22
Copy link
Contributor

✅ Commitlint tests passed!

More Info
{
  "valid": true,
  "errors": [],
  "warnings": [],
  "input": "feat(tools): knobs"
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AT passed Automated testing has passed demo Updating demo pages functionality Functionality, typically pertaining to the JavaScript. tools Development and build tools work in progress POC / Not ready for review
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

4 participants