fix(web): Zurueck im Album fuehrt dorthin zurueck, wo man geklickt hat
album-back-btn war die einzige Detailseite, die per navigate() VORWAERTS auf die Albenliste sprang statt einen Schritt im Verlauf zurueck. Dadurch landete man nicht dort, wo man das Album angeklickt hatte (z. B. Home), und es kam ein zusaetzlicher History-Eintrag auf den Stapel, durch den sich der Browser-Zurueck- Knopf erst arbeiten musste. Jetzt _navGoBack() wie bei Artist, Genre, Deezer- und Partial-Album; die Albenliste bleibt nur das Ziel, wenn es keinen Verlauf gibt. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
1 parent
eee98d193f
commit
39d47b78d4
1 file changed
+6
-1
+6
-1
@@ -572,8 +572,13 @@ export async function loadPartialAlbumDetail({ nd, deezer }) {
|
|||||||
// ─── ALBUM DETAIL ───
|
// ─── ALBUM DETAIL ───
|
||||||
export async function loadAlbumDetail(album) {
|
export async function loadAlbumDetail(album) {
|
||||||
if (S.cfg.showRomaji) await ensureArtistSortMap().catch(() => {});
|
if (S.cfg.showRomaji) await ensureArtistSortMap().catch(() => {});
|
||||||
|
// Zurück heißt zurück: einen Eintrag im Verlauf zurückgehen, wie auf jeder anderen Detailseite.
|
||||||
|
// Vorher navigierte dieser Knopf VORWÄRTS auf die Albenliste (bzw. den Artist) — das landete nicht
|
||||||
|
// dort, wo man das Album angeklickt hatte, und schob zusätzlich einen neuen History-Eintrag auf den
|
||||||
|
// Stapel, sodass der Browser-Zurück-Knopf sich durch diesen Müll arbeiten musste statt nach Home.
|
||||||
|
// Nur wenn es keinen Verlauf gibt (Direkteinstieg auf ein Album), bleibt die Albenliste das Ziel.
|
||||||
document.getElementById('album-back-btn').onclick = () =>
|
document.getElementById('album-back-btn').onclick = () =>
|
||||||
album._fromArtist ? navigate('artist-detail', album._fromArtist) : navigate('albums');
|
S._navCursor > 0 ? _navGoBack() : navigate('albums');
|
||||||
document.getElementById('album-detail-name').textContent = dispAlbumName(album);
|
document.getElementById('album-detail-name').textContent = dispAlbumName(album);
|
||||||
const artistEl = document.getElementById('album-detail-artist');
|
const artistEl = document.getElementById('album-detail-artist');
|
||||||
artistEl.innerHTML = parseArtists(album)
|
artistEl.innerHTML = parseArtists(album)
|
||||||
|
|||||||
Reference in new issue
Block a user