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

Fix check to display Autocomplete #3963

Merged
merged 1 commit into from
Jan 11, 2024
Merged

Fix check to display Autocomplete #3963

merged 1 commit into from
Jan 11, 2024

Conversation

GMartigny
Copy link
Contributor

Display Autocomplete list if footer slot is registered

Proposed Changes

  • Add a check for footer slot in Autocomplete list display condition

Display Autocomplete list if footer slot is registered
Copy link

netlify bot commented Jan 10, 2024

Deploy Preview for buefy-org ready!

Name Link
🔨 Latest commit 0699e08
🔍 Latest deploy log https://app.netlify.com/sites/buefy-org/deploys/659e6aa962f60a0008919803
😎 Deploy Preview https://deploy-preview-3963--buefy-org.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.

@kikuomax kikuomax self-assigned this Jan 11, 2024
@kikuomax
Copy link
Collaborator

kikuomax commented Jan 11, 2024

@GMartigny I tested the behavior with the following code:

<template>
    <div class="container is-max-desktop">
        <b-field label="Only footer">
            <b-autocomplete :data="[]">
                <template #footer>
                    <a><span>Add new...</span></a>
                </template>
            </b-autocomplete>
        </b-field>
    </div>
</template>

Without your PR: no dropdown showed up

With your PR: the dropdown with "Add new..." footer showed up

Thanks for your PR! Please attach any reproduction code next time.

Copy link
Collaborator

@kikuomax kikuomax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@kikuomax kikuomax merged commit ec2dfe9 into buefy:dev Jan 11, 2024
5 checks passed
kikuomax pushed a commit that referenced this pull request Feb 9, 2024
- Fixes the issue that no dropdown showed up if the data array was empty
  and only the footer slot was registered.
  Display Autocomplete list if footer slot is registered
kikuomax pushed a commit to kikuomax/buefy that referenced this pull request Feb 11, 2024
- Fixes the issue that no dropdown showed up if the data array was empty
  and only the footer slot was registered.
  Display Autocomplete list if footer slot is registered
kikuomax pushed a commit to ntohq/buefy-next that referenced this pull request Feb 12, 2024
- Fixes the issue that no dropdown showed up if the data array was empty
  and only the footer slot was registered.
  Display Autocomplete list if footer slot is registered
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Completed ✅
Development

Successfully merging this pull request may close these issues.

None yet

2 participants