Skip to content

Conversation

@Tofandel
Copy link
Contributor

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Documentation
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the dev branch (or v[X] branch)
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
  • It's been tested on a Cordova (iOS, Android) app
  • It's been tested on an Electron app
  • Any necessary documentation has been added or updated in the docs or explained in the PR's description.

Other information:
This issue is particularly problematic when using tooltips and menus, which need to be in the slot when rendering

Doesn't work

<QBadge label="Foo"><QMenu>Doesn't show</QMenu></QBadge>

Works

<QBadge>Foo<QMenu>Shows</QMenu></QBadge>

Tofandel and others added 3 commits May 18, 2022 22:01
This is particularly problematic when using

```vue
<QBadge label="Foo"><QMenu>Doesn't show</QMenu></QBadge>
```
@rstoenescu rstoenescu merged commit 026bb6f into quasarframework:dev May 22, 2022
@rstoenescu
Copy link
Member

Thanks for contributing!
This will go into 2.7.1

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.

2 participants