ARC-3792 object grid (without repeat blocks migration) - #380
Conversation
| // block) into an ie-objects search API request body. Lives in the config so the client's | ||
| // own url-filter-mapping logic (used by its search page) can be reused here, without the | ||
| // admin-core needing to depend on the client package. | ||
| clientSearchUrlToApiSearchUrl: (searchQuery: string) => IeObjectsSearchBody; |
There was a problem hiding this comment.
Ik vermoed dat je dit weer gaat moeten enablen in de client ook. Die PR was reeds gemerged, maar kon niet builden met die logica erin. Dus die method is in de client in commentaar gezet:
https://github.com/viaacode/hetarchief-client/blob/337147ef288338079be3929d37b73947dd562506/src/modules/admin/wrappers/admin-core-config.tsx#L221
| display: flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| background-color: colors.$color-gray-900; | ||
|
|
||
| // Fixed waveform look for audio, which has no thumbnail. | ||
| background-image: repeating-linear-gradient( | ||
| 90deg, | ||
| colors.$color-gray-400 0, | ||
| colors.$color-gray-400 2px, | ||
| transparent 2px, | ||
| transparent 8px | ||
| ); |
There was a problem hiding this comment.
Dat zijn die nieuwe waveforms? Daar hebben we een apart ticket voor. Ik zou voorstellen de nieuwe volledig in een aparte component te steken zodat we die kunnen hergebruiken. Enerzijds met een vaste achtergrondkleur, anderzijds met de optie de achtergrondkleur te randomizen. Al zou ik prefereren dat we dat allemaal in dat aparte ticket steken en voor nu de oude audiowave gebruiken
There was a problem hiding this comment.
we kunnen die replacen eens we die waveforms als aparte component implementeren
anders doen we nu dubbel werk met die er terug uit te halen
There was a problem hiding this comment.
Misschien dat deze beter verhuist naar de shared?
There was a problem hiding this comment.
waar gaan we die nog nodig hebben?
als het enkel door deze blok wordt gebruikt lijkt me dat hier beter
van zodra een andere blok die nodig heeft, kunnen we die verhuizen
| return ( | ||
| <section | ||
| className={clsx('c-block-objects-grid', className)} | ||
| style={backgroundColor ? { backgroundColor } : undefined} |
There was a problem hiding this comment.
Wat met de speciale gevallen zoals meemoo logo of zwart-wit?
There was a problem hiding this comment.
ik zie het nut niet van alle blokken te voorzien op die speciale cases, als meemoo die zowiezo niet gaat gebruiken
dat is wasted dev time
dan kijken we beter om die edge case background colors enkel toe te laten bij de specifieke blokken waar ze nuttig zijn
https://meemoo.atlassian.net/browse/ARC-3792 Extracts object format to type mapping, icon mapping, object detail path generation, UI breakpoints, and spacing variables into shared modules. This improves consistency, reusability, and maintainability across components like the objects grid and theme reels.
…-3792-objects-grid-block-2
2 participants
jira ticket:
https://meemoo.atlassian.net/browse/ARC-3792
oude pr (met repeat block migration code)
#362
example page:
http://localhost:3200/admin/content-paginas/af27aac8-40e9-4503-bf91-34892f49fc12
Adds the new
ObjectsGridcontent block, allowing users to display a dynamic grid of media objects:clientSearchUrlToApiSearchUrlconfiguration service.