feat: Integrate FoF Bookmarks support into Avocado theme - #195
Merged
Conversation
ram0ng1
commented
Aug 9, 2026
Owner
- Refactor bookmark handling to support both Avocado's native bookmarks and FoF Bookmarks.
- Update ThreadCard component to conditionally render bookmark actions based on the active bookmark system.
- Implement new bookmark utility functions to manage interactions with both bookmark systems.
- Create a paginated state for bookmarked posts, aligning with FoF Bookmarks functionality.
- Enhance the bookmarks API to expose new attributes and filters for discussions.
- Add styles for bookmarks in the sidebar and post controls, ensuring a cohesive UI.
- Update localization files to reflect changes in bookmark terminology and functionality.
- Introduce a new BookmarksRoute class to manage routing conflicts with FoF Bookmarks.
- Refactor bookmark handling to support both Avocado's native bookmarks and FoF Bookmarks. - Update ThreadCard component to conditionally render bookmark actions based on the active bookmark system. - Implement new bookmark utility functions to manage interactions with both bookmark systems. - Create a paginated state for bookmarked posts, aligning with FoF Bookmarks functionality. - Enhance the bookmarks API to expose new attributes and filters for discussions. - Add styles for bookmarks in the sidebar and post controls, ensuring a cohesive UI. - Update localization files to reflect changes in bookmark terminology and functionality. - Introduce a new BookmarksRoute class to manage routing conflicts with FoF Bookmarks.
Contributor
🔒 Regressão de segurança (Semgrep diff vs
|
Contributor
📦 Bundle size✅ Sem regressão de tamanho — gzip total −2.9 KB (-4.0%).
gzip nível 9. 🟢 = menor que o master · 🔴 = maior. Report-only — não bloqueia o merge. |
Contributor
🔬 Performance benchmarkHome (
|
| Métrica | PR | Master (baseline) | Δ |
|---|---|---|---|
| Performance (score) | 82 | 82 | (=) |
| FCP | 1613 ms | 1738 ms | 🟢 -125 ms (-7.2%) |
| LCP | 1976 ms | 2007 ms | 🟢 -31 ms (-1.5%) |
| TBT | 120 ms | 63 ms | 🔴 +57 ms (90.5%) |
| CLS | 0.000 | 0.000 | (=) |
| Speed Index | 1613 ms | 1738 ms | 🟢 -125 ms (-7.2%) |
| TTI | 1978 ms | 2009 ms | 🟢 -31 ms (-1.5%) |
💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)
- Sem compressão de texto — economia ~960 ms · ~1175 KB
Habilite gzip/brotli no servidor (nginx:gzip on; gzip_types text/css application/javascript;oubrotli on; brotli_types ...). Esse é um setting de host, não da extensão, mas reportar aqui ajuda. - JavaScript não utilizado — economia ~680 ms · ~549 KB
Useimport()dinâmico para componentes só usados em rotas específicas (TeamPage, AvocadoSearchPage). O webpack faz split automático se você usarimport("./components/TeamPage")dentro doroutesao invés de import estático no topo doindex.tsx. - CSS não utilizado — economia ~480 ms · ~584 KB
O bundleforum.csscarrega tudo deless/forum/**. Divida por rota (Home/Discussion/Tag) viaExtend\Frontend->css()condicional no JS, ou use PurgeCSS no webpack para o build de produção. - Recursos bloqueando o render — economia ~480 ms
Mova CSS não-crítico para<link rel="preload">ou injete inline o CSS above-the-fold doforum.less. Em Flarum 2, oforum.cssé servido como bloqueante por padrão — consideremedia="print" onloadpara folhas não-críticas (ex.: dark theme). - Cache HTTP curto — 8 resources found
Os assets versionados em/assets/forum-<hash>.jsdeveriam terCache-Control: public, max-age=31536000, immutable. Configure no nginx/apache, não no Flarum.
/discussions
| Métrica | PR | Master (baseline) | Δ |
|---|---|---|---|
| Performance (score) | 86 | 86 | (=) |
| FCP | 1504 ms | 1499 ms | 🔴 +5 ms (0.3%) |
| LCP | 1811 ms | 1806 ms | 🔴 +4 ms (0.2%) |
| TBT | 1 ms | 0 ms | 🔴 +1 ms |
| CLS | 0.000 | 0.000 | (=) |
| Speed Index | 1504 ms | 1499 ms | 🔴 +5 ms (0.3%) |
| TTI | 1811 ms | 1806 ms | 🔴 +4 ms (0.2%) |
💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)
- Sem compressão de texto — economia ~920 ms · ~1178 KB
Habilite gzip/brotli no servidor (nginx:gzip on; gzip_types text/css application/javascript;oubrotli on; brotli_types ...). Esse é um setting de host, não da extensão, mas reportar aqui ajuda. - JavaScript não utilizado — economia ~530 ms · ~574 KB
Useimport()dinâmico para componentes só usados em rotas específicas (TeamPage, AvocadoSearchPage). O webpack faz split automático se você usarimport("./components/TeamPage")dentro doroutesao invés de import estático no topo doindex.tsx. - Recursos bloqueando o render — economia ~520 ms
Mova CSS não-crítico para<link rel="preload">ou injete inline o CSS above-the-fold doforum.less. Em Flarum 2, oforum.cssé servido como bloqueante por padrão — consideremedia="print" onloadpara folhas não-críticas (ex.: dark theme). - CSS não utilizado — economia ~480 ms · ~590 KB
O bundleforum.csscarrega tudo deless/forum/**. Divida por rota (Home/Discussion/Tag) viaExtend\Frontend->css()condicional no JS, ou use PurgeCSS no webpack para o build de produção. - Cache HTTP curto — 9 resources found
Os assets versionados em/assets/forum-<hash>.jsdeveriam terCache-Control: public, max-age=31536000, immutable. Configure no nginx/apache, não no Flarum.
/tags
| Métrica | PR | Master (baseline) | Δ |
|---|---|---|---|
| Performance (score) | 87 | 87 | (=) |
| FCP | 1501 ms | 1498 ms | 🔴 +4 ms (0.2%) |
| LCP | 1763 ms | 1758 ms | 🔴 +6 ms (0.3%) |
| TBT | 0 ms | 0 ms | (=) |
| CLS | 0.000 | 0.000 | (=) |
| Speed Index | 1501 ms | 1498 ms | 🔴 +4 ms (0.2%) |
| TTI | 1763 ms | 1758 ms | 🔴 +6 ms (0.3%) |
💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)
- Sem compressão de texto — economia ~920 ms · ~1175 KB
Habilite gzip/brotli no servidor (nginx:gzip on; gzip_types text/css application/javascript;oubrotli on; brotli_types ...). Esse é um setting de host, não da extensão, mas reportar aqui ajuda. - JavaScript não utilizado — economia ~520 ms · ~579 KB
Useimport()dinâmico para componentes só usados em rotas específicas (TeamPage, AvocadoSearchPage). O webpack faz split automático se você usarimport("./components/TeamPage")dentro doroutesao invés de import estático no topo doindex.tsx. - Recursos bloqueando o render — economia ~520 ms
Mova CSS não-crítico para<link rel="preload">ou injete inline o CSS above-the-fold doforum.less. Em Flarum 2, oforum.cssé servido como bloqueante por padrão — consideremedia="print" onloadpara folhas não-críticas (ex.: dark theme). - CSS não utilizado — economia ~480 ms · ~588 KB
O bundleforum.csscarrega tudo deless/forum/**. Divida por rota (Home/Discussion/Tag) viaExtend\Frontend->css()condicional no JS, ou use PurgeCSS no webpack para o build de produção. - Cache HTTP curto — 8 resources found
Os assets versionados em/assets/forum-<hash>.jsdeveriam terCache-Control: public, max-age=31536000, immutable. Configure no nginx/apache, não no Flarum.
🥑 Dicas gerais para acelerar o tema (Flarum 2 + Avocado)
- Build de produção minificado — confirme que
npm run buildrodou commode: production(já é o caso nojs/package.json). Webpack tree-shakesimport { x } from "flarum/..."se o consumo for explícito. - Split por rota —
index.tsximporta todos os Components no topo. Trocar paraimport()dinâmico nas rotas raramente acessadas (TeamPage, AvocadoSearchPage, TagsPage) reduz oforum.jsinicial. - Less crítico inline — Flarum 2 serve
forum.cssbloqueando. Considere extrair o CSS above-the-fold (header + 1ª thread visível) e injetar inline viaExtend\Frontend->content(InlineCriticalCss::class). - Avatares — sirva via
<img loading="lazy">em todos os ThreadCards/PostCards que não estiverem no fold inicial. <link rel="preconnect">— se você usa CDN para fontes ou avatares S3, adicione preconnects no header.Extend\Frontend->js()rodam síncronos — todo arquivojs/dist/forum.jsé parseado no boot. Cadaextend()noindex.tsxroda antes da primeira pintura.- opcache + view cache — no host:
opcache.enable=1,opcache.validate_timestamps=0em produção, ephp flarum cache:clearno deploy. - HTTP/2 + Brotli no host — o Flarum gera bundles grandes; sem brotli você paga em transferência.
Lighthouse desktop, 1 run por URL. 🟢 = melhorou vs master · 🔴 = regrediu.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.