Skip to content

feat(core): adding isEmpty, hasSlotted w/ default slot - #2604

Merged
brianferry merged 7 commits into
mainfrom
feat/core-default-slot
Dec 4, 2023
Merged

feat(core): adding isEmpty, hasSlotted w/ default slot#2604
brianferry merged 7 commits into
mainfrom
feat/core-default-slot

Conversation

@brianferry

Copy link
Copy Markdown
Collaborator

Linked Issue: #2603

What I did

  1. Added an isEmpty function that can be called either empty (default slot) or with a slot name. This will return if the slot does not have content.
  2. hasSlotted now returns default slot if not slot is provided.

@changeset-bot

changeset-bot Bot commented Oct 10, 2023

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4f86242

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@patternfly/pfe-core Minor

Not sure what this means? Click here to learn what changesets are.

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

@brianferry
brianferry enabled auto-merge (squash) October 10, 2023 17:36
@brianferry brianferry linked an issue Oct 10, 2023 that may be closed by this pull request
@netlify

netlify Bot commented Oct 10, 2023

Copy link
Copy Markdown

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit 8b82439
😎 Deploy Preview https://deploy-preview-2604--patternfly-elements.netlify.app/

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions Bot added the AT passed Automated testing has passed label Oct 10, 2023
Comment thread core/pfe-core/controllers/slot-controller.ts
Comment thread core/pfe-core/controllers/slot-controller.ts
@bennypowers bennypowers added this to the PatternFly Elements 3 milestone Nov 8, 2023
@eyevana
eyevana self-requested a review November 14, 2023 19:09
Comment thread core/pfe-core/controllers/slot-controller.ts
Comment thread .changeset/cold-cars-relate.md Outdated
@brianferry
brianferry merged commit ac0c376 into main Dec 4, 2023
@brianferry
brianferry deleted the feat/core-default-slot branch December 4, 2023 17:46
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 ready to merge

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[core] Add default slot getter to SlotController

2 participants