cleaning up buttons
This commit is contained in:
parent
62c7fb9e19
commit
04acbf896e
|
|
@ -14,7 +14,6 @@
|
||||||
* @param {boolean} config.isSelected - Whether track is selected
|
* @param {boolean} config.isSelected - Whether track is selected
|
||||||
* @param {boolean} config.isCached - Whether track is cached locally
|
* @param {boolean} config.isCached - Whether track is cached locally
|
||||||
* @param {boolean} config.isActive - Whether this is the currently playing track
|
* @param {boolean} config.isActive - Whether this is the currently playing track
|
||||||
* @param {boolean} config.showPlayButton - Show play button (queue only)
|
|
||||||
* @param {boolean} config.draggable - Whether element is draggable
|
* @param {boolean} config.draggable - Whether element is draggable
|
||||||
* @returns {HTMLElement}
|
* @returns {HTMLElement}
|
||||||
*/
|
*/
|
||||||
|
|
@ -26,7 +25,6 @@
|
||||||
isSelected,
|
isSelected,
|
||||||
isCached,
|
isCached,
|
||||||
isActive,
|
isActive,
|
||||||
showPlayButton,
|
|
||||||
draggable
|
draggable
|
||||||
} = config;
|
} = config;
|
||||||
|
|
||||||
|
|
@ -53,8 +51,6 @@
|
||||||
// Build HTML
|
// Build HTML
|
||||||
const checkmark = isSelected ? '<span class="track-checkmark">✓</span>' : '';
|
const checkmark = isSelected ? '<span class="track-checkmark">✓</span>' : '';
|
||||||
const trackNum = displayIndex != null ? `<span class="track-number">${displayIndex}.</span>` : '';
|
const trackNum = displayIndex != null ? `<span class="track-number">${displayIndex}.</span>` : '';
|
||||||
const playBtn = showPlayButton ? '<button class="track-play-btn" title="Play">▶</button>' : '';
|
|
||||||
const previewBtn = '<button class="track-preview-btn" title="Preview">⏵</button>';
|
|
||||||
|
|
||||||
div.innerHTML = `
|
div.innerHTML = `
|
||||||
${checkmark}
|
${checkmark}
|
||||||
|
|
@ -62,8 +58,6 @@
|
||||||
${trackNum}
|
${trackNum}
|
||||||
<span class="track-title">${escapeHtml(title)}</span>
|
<span class="track-title">${escapeHtml(title)}</span>
|
||||||
<span class="track-actions">
|
<span class="track-actions">
|
||||||
${playBtn}
|
|
||||||
${previewBtn}
|
|
||||||
<span class="duration">${M.fmt(track.duration)}</span>
|
<span class="duration">${M.fmt(track.duration)}</span>
|
||||||
</span>
|
</span>
|
||||||
`;
|
`;
|
||||||
|
|
|
||||||
|
|
@ -117,7 +117,6 @@
|
||||||
isSelected,
|
isSelected,
|
||||||
isCached,
|
isCached,
|
||||||
isActive,
|
isActive,
|
||||||
showPlayButton: type === 'queue',
|
|
||||||
draggable: isDraggable
|
draggable: isDraggable
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|
@ -269,24 +268,6 @@
|
||||||
render();
|
render();
|
||||||
};
|
};
|
||||||
|
|
||||||
// Play button (queue only)
|
|
||||||
const playBtn = div.querySelector('.track-play-btn');
|
|
||||||
if (playBtn) {
|
|
||||||
playBtn.onclick = (e) => {
|
|
||||||
e.stopPropagation();
|
|
||||||
playTrack(track, originalIndex);
|
|
||||||
};
|
|
||||||
}
|
|
||||||
|
|
||||||
// Preview button
|
|
||||||
const previewBtn = div.querySelector('.track-preview-btn');
|
|
||||||
if (previewBtn) {
|
|
||||||
previewBtn.onclick = (e) => {
|
|
||||||
e.stopPropagation();
|
|
||||||
previewTrack(track);
|
|
||||||
};
|
|
||||||
}
|
|
||||||
|
|
||||||
// Context menu
|
// Context menu
|
||||||
div.oncontextmenu = (e) => {
|
div.oncontextmenu = (e) => {
|
||||||
e.preventDefault();
|
e.preventDefault();
|
||||||
|
|
@ -834,6 +815,8 @@
|
||||||
|
|
||||||
const sub = document.createElement("div");
|
const sub = document.createElement("div");
|
||||||
sub.className = "context-submenu";
|
sub.className = "context-submenu";
|
||||||
|
const subInner = document.createElement("div");
|
||||||
|
subInner.className = "context-submenu-inner";
|
||||||
item.submenu.forEach(subItem => {
|
item.submenu.forEach(subItem => {
|
||||||
const subEl = document.createElement("div");
|
const subEl = document.createElement("div");
|
||||||
subEl.className = "context-menu-item";
|
subEl.className = "context-menu-item";
|
||||||
|
|
@ -843,8 +826,9 @@
|
||||||
hideContextMenu();
|
hideContextMenu();
|
||||||
subItem.action();
|
subItem.action();
|
||||||
};
|
};
|
||||||
sub.appendChild(subEl);
|
subInner.appendChild(subEl);
|
||||||
});
|
});
|
||||||
|
sub.appendChild(subInner);
|
||||||
el.appendChild(sub);
|
el.appendChild(sub);
|
||||||
} else if (!item.disabled) {
|
} else if (!item.disabled) {
|
||||||
el.onclick = () => {
|
el.onclick = () => {
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue