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

docs(guide/components/slots): clarify slot presence phrasing #3124

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

alexchexes
Copy link
Contributor

Description of Problem

The original phrasing "whether or not a slot is present" was potentially misleading, as it suggested checking the existence of the slot itself rather than whether content was passed to it. Updated the explanation to clarify this distinction for better readability and accuracy.

Proposed Solution

The original phrase was replaced with "...whether or not content has been passed to a slot"

Additional Information

The original phrasing "whether or not a slot is present" was potentially misleading, as it suggested checking the existence of the slot itself rather than whether content was passed to it. Updated the explanation to clarify this distinction for better readability and accuracy.
Copy link

netlify bot commented Dec 4, 2024

Deploy Preview for vuejs ready!

Name Link
🔨 Latest commit 6734471
🔍 Latest deploy log https://app.netlify.com/sites/vuejs/deploys/675177ffe85eff0008f7119f
😎 Deploy Preview https://deploy-preview-3124--vuejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@alexchexes
Copy link
Contributor Author

alexchexes commented Dec 4, 2024

Oh, I just noticed another related phrase right below the one I addressed in this PR:

When the header / footer / default is present we want to wrap them to provide additional styling:

The phrase "When the header/footer/default is present..." could also be interpreted as referring to the slot's definition, rather than whether content is provided for the slot. A clearer alternative:

When content for the header / footer / default is present, we want to wrap it to provide additional styling:

Let me know if I should open another PR for this.

@AloisSeckar
Copy link
Contributor

You shouldn't open a new PR. Make a new commit into this one. The proposed change is adressing the same thing.

The phrase "When the header/footer/default is present..." could be interpreted as referring to the slot's definition, rather than whether content is provided for the slot
@alexchexes
Copy link
Contributor Author

alexchexes commented Dec 5, 2024

Make a new commit into this one

done. Thanks

@NataliaTepluhina NataliaTepluhina merged commit c1af459 into vuejs:main Dec 9, 2024
5 checks passed
szymonlicau pushed a commit to monterail/vue3-docs-pl that referenced this pull request Dec 9, 2024
)

* docs(guide/components/slots): clarify slot presence phrasing

The original phrasing "whether or not a slot is present" was potentially misleading, as it suggested checking the existence of the slot itself rather than whether content was passed to it. Updated the explanation to clarify this distinction for better readability and accuracy.

* docs(guide/components/slots): clarify slot presence phrasing (2)

The phrase "When the header/footer/default is present..." could be interpreted as referring to the slot's definition, rather than whether content is provided for the slot
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.

3 participants