Enhance post badge position settings and styles - #193
Merged
Conversation
ram0ng1
commented
Aug 5, 2026
Owner
- Updated the help text for the post badge position setting to clarify the functionality of each option, including the default Flarum badge behavior.
- Added a new default option label for the post badge position in both English and Portuguese locales.
- Modified the CSS classes in PostBadges.less to ensure proper styling based on the selected badge position, including the introduction of a custom class for better specificity.
- Adjusted the JavaScript logic to toggle the custom class based on the selected badge position, ensuring the correct styles are applied across different layouts.
- Updated the help text for the post badge position setting to clarify the functionality of each option, including the default Flarum badge behavior. - Added a new default option label for the post badge position in both English and Portuguese locales. - Modified the CSS classes in PostBadges.less to ensure proper styling based on the selected badge position, including the introduction of a custom class for better specificity. - Adjusted the JavaScript logic to toggle the custom class based on the selected badge position, ensuring the correct styles are applied across different layouts.
…/ram0ng1/avocado into refatoracao-tokens-design-system
Contributor
📦 Bundle size🟡 Crescimento pequeno — gzip total +0.1 KB (+0.1%), abaixo do limiar de regressão (2% / 2 KB).
gzip nível 9. 🟢 = menor que o master · 🔴 = maior. Report-only — não bloqueia o merge. |
Contributor
🔒 Regressão de segurança (Semgrep diff vs
|
Contributor
🔬 Performance benchmarkHome (
|
| Métrica | PR | Master (baseline) | Δ |
|---|---|---|---|
| Performance (score) | 82 | 85 | 🔴 -3pp (-3.5%) |
| FCP | 1756 ms | 1542 ms | 🔴 +215 ms (13.9%) |
| LCP | 1983 ms | 1903 ms | 🔴 +80 ms (4.2%) |
| TBT | 62 ms | 52 ms | 🔴 +10 ms (19.2%) |
| CLS | 0.000 | 0.000 | (=) |
| Speed Index | 1756 ms | 1542 ms | 🔴 +215 ms (13.9%) |
| TTI | 1985 ms | 1906 ms | 🔴 +79 ms (4.1%) |
💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)
- Sem compressão de texto — economia ~920 ms · ~1161 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 · ~536 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 · ~582 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 ~440 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 | 1505 ms | 1501 ms | 🔴 +4 ms (0.3%) |
| LCP | 1811 ms | 1809 ms | 🔴 +2 ms (0.1%) |
| TBT | 0 ms | 1 ms | 🟢 -1 ms (-100.0%) |
| CLS | 0.000 | 0.000 | (=) |
| Speed Index | 1505 ms | 1501 ms | 🔴 +4 ms (0.3%) |
| TTI | 1811 ms | 1809 ms | 🔴 +2 ms (0.1%) |
💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)
- Sem compressão de texto — economia ~920 ms · ~1167 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 · ~562 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 ~520 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. - 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). - 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 | 1506 ms | 🟢 -5 ms (-0.3%) |
| LCP | 1761 ms | 1772 ms | 🟢 -11 ms (-0.6%) |
| TBT | 0 ms | 0 ms | (=) |
| CLS | 0.000 | 0.000 | (=) |
| Speed Index | 1501 ms | 1506 ms | 🟢 -5 ms (-0.3%) |
| TTI | 1761 ms | 1772 ms | 🟢 -11 ms (-0.6%) |
💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)
- Sem compressão de texto — economia ~920 ms · ~1161 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 · ~566 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 · ~587 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.