openfoodfacts/openfoodfacts-server

Folksonomy Engine loads CSS and a font from CDNs

Open

#9.012 aberto em 17 de set. de 2023

Ver no GitHub
 (6 comments) (3 reactions) (0 assignees)Perl (236 forks)batch import
:star: :bug: top bug:star: top issueCSSJavaScriptgood first issue🎯 P1🏷️ Folksonomy Project

Métricas do repositório

Stars
 (477 stars)
Métricas de merge de PR
 (Mesclagem média 15d 21h) (157 fundiu PRs em 30d)

Description

https://github.com/openfoodfacts/openfoodfacts-server/blob/a0e358e7a79f55ed457d5676067e121ff7097416/html/js/folksonomy.js#L63-L65

image

For ProductOpener itself, we usually try to reduce the dependency on 3rd party CDNs. folksonomy.js does the opposite by loading three files from three different CDNs.

  1. jQuery UI is already bundled with ProductOpener
  2. FontAwesome could be revisited, because ProductOpener now mostly seems to use Material Icons since the redesign
  3. Bundle jqGrid, or replace it (if there's feature parity) with other grid already bundled?

Code pointers

You can find some code pointers here. If some are missing, feel free to add them yourself in this file, or to ask them.

Part of

  • #7525

Guia do colaborador

Direção de pesquisa
Examine o uso de jqGrid e jQuery UI no folksonomy.js, identifique quais recursos são realmente usados e determine se podem ser substituídos por bibliotecas já incluídas (ex.: jQuery UI) ou código personalizado. Proponha um plano para remover dependências de CDN para CSS e fontes, possivelmente substituindo FontAwesome por Material Icons.
Pilha de tecnologia
javascripthtmlcss
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJavaScriptHTMLCSS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65