Enlaces de encabezado personalizados (iconos)

He actualizado esto y veo el siguiente mensaje de error en la página /admin/customize/themes:

Error: Variable no definida: "$add-whitespace". en la línea 14 de common.scss >> @if $add_whitespace == "true" { ----^

Actualización: marqué la nueva casilla “whitespace” y la desmarqué (guardando en cada ocasión) y el error ha desaparecido.

2 Me gusta

El informe no es incorrecto, pero la evaluación de la gravedad está un poco fuera de lugar. Estos enlaces no son contenido generado por el usuario. Solo los administradores pueden agregarlos desde el panel de administración. Por lo tanto, es seguro asumir que, como administrador, no enlazarás sitios en los que no confíes.

Dicho esto, tuve una breve charla con David y sugirió agregar rel="noopener" a los enlaces externos de todos modos. Lo he añadido aquí. Si un enlace está configurado para abrirse en una nueva pestaña, se agrega automáticamente.

En cuanto a noreferrer,

Creo que es más beneficioso no añadirlo, ya que es posible que quieras enlazar tu sitio principal y obtener análisis sobre cuántos usuarios son redirigidos al sitio principal desde tu comunidad.

¿Sigue siendo así? ¿Puedes compartir una captura de pantalla de la configuración de tu componente?

Sí, yo también lo noté al actualizar, pero no está relacionado con los nuevos cambios. De alguna manera, la nueva configuración no es detectada inmediatamente por el compilador. Por lo tanto, es posible que necesites marcar y desmarcar esa configuración.

Aunque no te preocupes, según mis pruebas, el error no tiene ningún impacto en tu sitio.

5 Me gusta

Mi sitio me indicó hoy que debía actualizar el componente del tema, lo cual acabo de hacer haciendo clic en el botón de actualizar.
Pero ahora muestra el error:
Error: Variable no definida: "$add-whitespace". en la línea 14 de common.scss >> @if $add_whitespace == "true" { ----^

Yo tenía esto y se resolvió solo cuando marqué y desmarqué la casilla: Custom Header Links (icons) - #129 by Jonathan5

2 Me gusta

Gracias por el informe, @GuidoD.

Somos conscientes de este problema, pero no debería tener ningún impacto en tu sitio. Se trata de un problema del compilador que estamos siguiendo internamente.

Avísame si causa algún problema en tu sitio; no debería hacerlo.

2 Me gusta

Cierto, no noté ningún problema adicional.
Actualmente estoy realizando una reconstrucción del sitio en Discourse y lo verificaré de nuevo después. Si el error persiste, revisaré la sugerencia de Jonathan.

Después de la reconstrucción, el error desapareció :slight_smile:

Gracias por tu excelente componente de tema que ya utilizamos desde hace dos años.

2 Me gusta

Hola, estoy usando Discourse 2.8.0.beta1 y al actualizar a la última versión de este componente del tema, obtengo este mensaje de error:

Error: Variable no definida: "$add-whitespace". en la línea 14 de common.scss >> @if $add_whitespace == "true" { ----^
2 Me gusta

Tengo el mismo error. Estoy en la versión estable 2.7.

Lo entiendo. Como mencioné anteriormente, esto no afectará en absoluto tu sitio.

4 Me gusta

Puedo confirmar que esto funciona.

Marca esta opción, luego desmárcala y recarga la página.

screenshot-vapingcommunity.co.uk-2021.06.07-13_48_21

¡Hola, gracias por este plugin!
He realizado algunas observaciones sobre los iconos FA que me gustaría compartir:
La integración de los iconos far- parece funcionar (regular), pero los fas- no (sólido).

Puedes probar esto:

fas-building
vs.
far-building

¿Tienes alguna idea sobre esto? Hay muy pocos iconos far- en comparación con los fas- (por no hablar de la perspectiva de diseño), así que esto es un problema…

Creo que tienes que buscar en el código para ver qué iconos se pueden usar y qué nombres usar para ellos (a menos que haya cambiado desde el comentario anterior).

Bueno, la descripción del componente del tema sugiere que se puede usar cada icono gratuito de FA5, independientemente de su clase (brand, regular, solid).

Echa un vistazo al código al que te referiste:

No hay ninguna cláusula if para solid. Entonces debería decir:

when "solid"
  key = "fas-"

¿Verdad?

Por lo que puedo decir, esa afirmación siempre ha sido incorrecta.

De acuerdo, en este caso, ¿debería corregirse en la descripción del componente del tema? Pero en realidad funciona con todos los iconos far-, así que mi opinión es que los fas- simplemente se “olvidaron”.

1 me gusta

El límite de mi conocimiento es que no funciona como se describe, y cómo averiguar qué iconos funcionarán al examinar el código :slight_smile:

Creo que los nombres de las variables para los iconos sólidos generalmente se implementan sin prefijo. ¿Probaste simplemente building?

Sí, por supuesto. Sin éxito.
Supongo que solo un pequeño subconjunto de fas- se incluye con Discourse.

Funciona para mí.. hay un subconjunto incluido por defecto (puedes verlo en /styleguide/atoms/icons). Los iconos adicionales primero debes agregarlos, ya sea en la configuración de administración o en el componente mismo, como con este:

2 Me gusta

¡También logré que funcione (usando “building” en los iconos SVG y en el componente del tema) – muestra la versión sólida del icono :+1: Quizás tenga que comerme mis palabras (aún estoy casi seguro de que algunas cosas no funcionaron como deberían…)

2 Me gusta