Skip to content

Conversation

@daniel89fg
Copy link
Contributor

…diseño responsivo.

El widgetRadio permite usarlo en modo imágenes, pero el cls no lo tiene el core, lo tiene el plugin de PlantillasPDF, claro mientras este ese plugin activo siempre se ve bien, pero el widget es del core no del plugin, po lo tanto el css debe ser del core, no del plugin. Más que nada porque tanto PlantillasPDF, como MultiEmail usan dicho css y no tiene sentido que cada plugin que quiera usar el widgetRadio con imágenes deba cargar su css.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Copilot AI review requested due to automatic review settings January 5, 2026 09:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR moves the CSS styling for the radio widget from the PlantillasPDF plugin to the core, addressing the issue where the widget is part of core but its styling was only available when specific plugins were active. This ensures that any plugin using the image-based radio widget will have proper styling without duplicating CSS files.

  • Added CSS file with styling for image-based radio widgets including animations and responsive design
  • Includes visual feedback through opacity changes and checkmark overlays when radio options are selected
  • The widget implementation already references this CSS file path

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NeoRazorX NeoRazorX merged commit 69949ad into NeoRazorX:master Jan 5, 2026
28 checks passed
@daniel89fg daniel89fg deleted the css-widgetRadio branch January 5, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants