Skip to content
This repository has been archived by the owner on Nov 30, 2020. It is now read-only.

feat(fab): add trailingIcon slot #411

Merged
merged 2 commits into from
Aug 27, 2019
Merged

Conversation

tychenjiajun
Copy link
Contributor

No description provided.

Copy link
Owner

@matsp matsp left a comment

Choose a reason for hiding this comment

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

Better solution!

@tychenjiajun tychenjiajun merged commit ea2697b into matsp:master Aug 27, 2019
@tychenjiajun tychenjiajun deleted the fab branch August 27, 2019 08:03
<slot />
<slot name="icon" />
<div
v-if="extended"
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems to have introduced a bug which makes it impossible to make an extended FAB. The default slot is only shown if extended is true, but extended is set to false unless there is text in the default slot.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahhh, I forgot to update readme. I'll open a new PR to fix it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants