Issue #731💬 RespondidoAbierto el 9 de enero de 2018por ryandebaReacciones 1

Los rasgos de tipo "color" no funcionan

Respuesta rápidapor artf1

¿Necesita el objeto Trait un método setValueFromInput? Esto lleva a la duplicación de código, pero sigo pensando que funcionará mejor '''js en Trait.js setValueFromInput(value, final = 1, opts = {}) { const toSet = { valor }; this.set(toSet, { ... opts, evitarAlmacenar: 1}); Tengo que activar el cambio si (final) { th...

Lee la respuesta completa abajo ↓

Pregunta

Hola @artf,

Ya había mencionado en una solicitud de tirada que los rasgos de tipo 'color' no funcionan. En ese momento no los necesitaba, pero ahora sí, así que me gustaría mirar cómo arreglarlo si no has empezado ya. El error está en el objeto 'InputColor' cuando llama a 'model.setValueFromInput'; los rasgos no tienen ese método, así que genera un error. ¿Qué crees que es la mejor manera de solucionar esto? ¿El objeto 'Trait' necesita un método 'setValueFromInput'? ¿O debería 'InputColor' cambiar para que sea lo suficientemente inteligente como para llamar diferentes métodos para un Rasgo frente a una Propiedad? ¿Quizá otra cosa?

Mis experimentos iniciales (improvisados) muestran que cambiar todas las líneas 'model.setValueFromInput' en 'InputColor' a esto lo hace bastante cerca de funcionar... No estoy seguro de si este es el camino correcto o no: 'model.setValueFromInput && model.setValueFromInput(cl, 0) || model.setValorObjetivoY & model.setValorObjetivo(cl);'

Respuestas (2)

artf9 de enero de 2018

¿Necesita el objeto Trait un método setValueFromInput?

Esto lleva a la duplicación de código, pero sigo pensando que funcionará mejor '''js en Trait.js setValueFromInput(value, final = 1, opts = {}) { const toSet = { valor }; this.set(toSet, { ... opts, evitarAlmacenar: 1});

Tengo que activar el cambio si (final) { this.set('value', '', opts); this.set(toSet, opts); } },

además, 'setTargetValue' no es realmente una forma correcta porque primero tienes que actualizar el modelo (un oyente actualizará el objetivo más adelante)

El verdadero problema es un mal diseño desde el principio. Para ser correcto, debería tener dentro de 'abstract/ui' un InputModel y su InputView (igual para otros tipos de entrada), y luego usarlos dentro de rasgos y estilos. Básicamente, la estructura tiene que estar más cerca de lo que se hace con las propiedades dentro del Style Manager, pero queda un poco más de trabajo por hacer -.-
  
lock[bot]18 de septiembre de 2019

Este hilo se ha bloqueado automáticamente porque no ha habido actividad reciente desde que se cerró. Por favor, abre un nuevo problema para bugs relacionados.

Preguntas y respuestas relacionadas

Continúa investigando con debates sobre temas similares.

Plugins de pago que cumplen con este problema

Seleccionado por temas clave y relevancia de etiquetas para ayudarte a enviar más rápido.

Ver todos los plugins

Cargando recomendaciones de plugins de pago...

Opción gratuita

Consulta los plugins de código abierto de GrapesJS en GitHub O haz una búsqueda rápida en nuestro catálogo gratuito.

Explora plugins gratuitos →
Opción premium

Los plugins premium incluyen soporte, actualizaciones regulares y funciones listas para producción — ahorrando días de trabajo de integración.

Explora plugins premium →

Tutoriales relacionados

Guías detalladas sobre el mismo tema.

Todos los tutoriales →

Explorar categorías de plugins

Ve directamente a las páginas de categorías de plugins en el marketplace.