Skip to content

Conversation

@brianferry
Copy link
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
Copy link

changeset-bot bot commented Oct 10, 2023

🦋 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
Copy link

netlify bot commented Oct 10, 2023

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
@bennypowers bennypowers added this to the PatternFly Elements 3 milestone Nov 8, 2023
@eyevana eyevana self-requested a review November 14, 2023 19:09
@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

3 participants