Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions DATA-MODEL.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@ public/data/

Lista leve com metadados básicos. Use este para listar cantorias sem carregar estrofes.

**⚠️ IMPORTANTE - Ordenação:** A ordem dos itens no array `cantorias` é **cronológica** (primeira = mais antiga, última = mais recente). Para exibir as mais recentes primeiro, inverta a ordem do array com `.reverse()`.

```json
{
"$schema": "https://json-schema.org/draft-07/schema#",
Expand Down
18 changes: 10 additions & 8 deletions view/src/routes/cantorias.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,21 +34,23 @@ function CantoriasPage() {
}

// Ordenação primária
if (sortBy === "alfabetica") {
if (sortBy === "recentes") {
// Inverte a ordem do array (último adicionado = mais recente)
filtered.reverse();
} else if (sortBy === "alfabetica") {
filtered.sort((a, b) => a.titulo.localeCompare(b.titulo));
} else if (sortBy === "estilo") {
filtered.sort((a, b) => a.estilo.nome.localeCompare(b.estilo.nome));
} else if (sortBy === "destaque") {
filtered.sort((a, b) => (b.destaque ? 1 : 0) - (a.destaque ? 1 : 0));
// Ordenação secundária: colocar cantorias COM vídeo primeiro apenas no modo destaque
filtered.sort((a, b) => {
const aTemVideo = a.links.youtube ? 1 : 0;
const bTemVideo = b.links.youtube ? 1 : 0;
return bTemVideo - aTemVideo; // Com vídeo primeiro
});
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Sort Order Conflict: Destaque Ignored.

The destaque sorting mode calls sort() twice sequentially, causing the second sort (video presence) to completely override the first sort (destaque flag). Items will only be sorted by video presence, ignoring the destaque property entirely. To fix, combine both criteria into a single comparator function with primary/secondary sort logic.

Fix in Cursor Fix in Web

}

// Ordenação secundária: sempre colocar cantorias COM vídeo primeiro
filtered.sort((a, b) => {
const aTemVideo = a.links.youtube ? 1 : 0;
const bTemVideo = b.links.youtube ? 1 : 0;
return bTemVideo - aTemVideo; // Com vídeo primeiro
});

return filtered;
}, [cantorias, filterEstilo, sortBy]);

Expand Down