Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Fix check to display Autocomplete#3963

Merged
kikuomax merged 1 commit intobuefy:devfromGMartigny:patch-1
Jan 11, 2024
Merged

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

netlifybot commentedJan 10, 2024
edited
Loading

Deploy Preview forbuefy-org ready!

NameLink
🔨 Latest commit0699e08
🔍 Latest deploy loghttps://app.netlify.com/sites/buefy-org/deploys/659e6aa962f60a0008919803
😎 Deploy Previewhttps://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 yourNetlify site configuration.

@kikuomaxkikuomax self-assigned thisJan 11, 2024
@kikuomax
Copy link
Collaborator

kikuomax commentedJan 11, 2024
edited
Loading

@GMartigny I tested the behavior with the following code:

<template>    <divclass="container is-max-desktop">        <b-fieldlabel="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.

GMartigny reacted with thumbs up emoji

Copy link
Collaborator

@kikuomaxkikuomax 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!

wesdevpro reacted with rocket emoji
@kikuomaxkikuomax merged commitec2dfe9 intobuefy:devJan 11, 2024
kikuomax pushed a commit that referenced this pull requestFeb 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 requestFeb 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 requestFeb 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 freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@kikuomaxkikuomaxkikuomax approved these changes

@wesdevprowesdevproAwaiting requested review from wesdevpro

Assignees

@kikuomaxkikuomax

Labels
None yet
Projects
No open projects
Status: Completed ✅
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@GMartigny@kikuomax

[8]ページ先頭

©2009-2025 Movatter.jp