- Issue created by @rfmarcelino
- 🇫🇷France lazzyvn paris
This patch is useless because you can do it in many ways.
for canvas width you can set width/height by custom theme css. or you can override twig with custom theme sign.html.twig.If you want to sign image just upload default image it will have width/height as default image
For reset button you can enable toolbox option you will have clear button.
Yes for ckeditor 5, it would be nice to configure width and reset button. In this patch I see you add these options to ckeditor 5 but I don't see any change in ckeditor5 plugin?
- 🇵🇹Portugal rfmarcelino
Hi @lazzyvn
Thank you for your work.Yes, you're right. I didn't notice that the toolbox also had it.
Maybe the help text could be adjusted to make ite clearer:
"Show toolbox to modify dot size, color pen or clear the canvas."I think the reset should be a separate option than the toolbox. The reset/clear should be useful to most users, but not necessarly the line width or color.
Regarding the CKEditor, to be honest I only tested the field widget.
Since you consider this useless, go ahead and close as designed. No point of further development.BTW, I added a new commit anyway to the MR to revert code added in the last release that caused an error:
'Drupal\Core\File\Exception\DirectoryNotReadyException'
It's not related to this issue, but since it's necessary for it to work... - 🇫🇷France lazzyvn paris
Check dev version if it work for separated option width and clear button
- 🇵🇹Portugal rfmarcelino
On the site I was working on, the width is working but I'm not seeing the clear button.
I tried on a blank install using simpletest.me and when adding the node with that field, I get a WSOD:
- eb2f5b80 committed on 8.x-1.x
Issue #3469743: Allow to set canva width and add reset button
- eb2f5b80 committed on 8.x-1.x
- Status changed to Fixed
4 months ago 3:56pm 23 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.