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(MdListItem): expand content cursor style and hover #1808

Merged
merged 2 commits into from
Jun 8, 2018

Conversation

Pwntus
Copy link
Contributor

@Pwntus Pwntus commented Jun 7, 2018

Fixes #1643

Default expanded list structure:

.md-list-item-container
  .md-list-item-content <-- hover, pointer if expand
  .md-list-expand
    .md-list
      .md-list-item
        .md-list-item-container
          .md-list-item-content <-- no hover, no pointer
  • Specifically target direct .md-list-item-content children of .md-list-item-container:not(.md-list-item-default) to use hover. This eliminates the whole expanded list getting hover-effects.
  • If .md-list-item-container is expand and :not(.md-list-item-default), apply hover to direct .md-list-item-content children, giving topmost list-element in expandable hover.
  • Specifically target direct md-list-expand-icon children when parent is .md-active.

May interfere with pending PR #1760

@Samuell1 Samuell1 merged commit 739cefa into vuematerial:dev Jun 8, 2018
marcosmoura added a commit that referenced this pull request Jun 9, 2018
* origin/dev:
  fix(MdListItem): expand content cursor style and hover (#1808)
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.

3 participants