Skip to content

Commit

Permalink
Fix unsupported button tooltips.
Browse files Browse the repository at this point in the history
The refactoring that allowed asset cards to reload their buttons
also ended up breaking the tooltips on unsupported buttons. This
fixes that bug.

Change-Id: Ib33845553de99c8a134529b003c3bd3d319d99bd
  • Loading branch information
theodab committed May 8, 2019
1 parent a90a8f9 commit 471d1a6
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion demo/asset_card.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@ class AssetCard {
this.actions_.classList.add('mdl-card__actions');
this.actions_.classList.add('mdl-card--border');
this.card_.appendChild(this.actions_);
this.remakeButtons();

const progressContainer = document.createElement('div');
this.progressBar_.classList.add('hidden');
Expand All @@ -86,6 +85,9 @@ class AssetCard {
this.card_.appendChild(progressContainer);

parentDiv.appendChild(this.card_);
// Remake buttons AFTER appending to parent div, so that any tooltips can
// be placed.
this.remakeButtons();
}

/**
Expand Down

0 comments on commit 471d1a6

Please sign in to comment.