vuematerial / vue-material

Vue.js Framework - ready-to-use Vue components with Material Design, free forever.
https://www.creative-tim.com/vuematerial
MIT License
9.88k stars 1.16k forks source link

[mdMenu] beta-12 release regression: auto-close menu content #2200

Closed okolomiets closed 4 years ago

okolomiets commented 4 years ago

Steps to reproduce

After being updated to beta-12 the MdMenu component started malfunctioning.

Regular usage as following:

<md-menu
      md-size="big"
      md-align-trigger
      md-close-on-click
      @md-opened="opened = true"
      @md-closed="opened = false">

        <md-button
          :class="[{ 'opened': opened }, isWorkspace ? 'md-primary' : isPlayback ? 'md-secondary' : 'md-error']"
          class="md-raised caret__button"
          md-menu-trigger>
          <span class="caret__button-label">
            {{ NAVIGATION_TITLE }}
            <icon
              name="chevron-down" />
          </span>
        </md-button>

      <md-menu-content> ... </md-menu-content>

</md-menu>

Which browser?

Chrome - 81.0.4044.122 (Official Build) (64-bit) Safari - 13.1 (15609.1.20.111.8)

What is expected?

The component should behave this way:

What is actually happening?

The issue looks like as following:

Confirmed: downgrading to beta-11 fixes the issue.

okolomiets commented 4 years ago

So after a quick investigation, I found out that problematic line is md-close-on-click which should be replaced with md-close-on-select since the fix from beta-12.

okolomiets commented 4 years ago

I'm closing the issue. This is not a regression rather proper behaviour.