- Issue created by @sachin.bansal27
- ๐ฎ๐ณIndia ayush.pandey
Tried the module with version 2.0.0 on Drupal9, and the issue is reproducible. The "summary help text" is working now with the fix in the patch.
- Status changed to Needs work
about 1 year ago 2:22am 28 September 2023 - ๐ช๐จEcuador jwilson3
The patch works and fixes a critical issue. The module is unusable on Drupal 10 without this.
However, it needs a little love...
+++ b/text_summary_options.module @@ -79,30 +79,30 @@ function text_summary_options_field_widget_form_alter(&$element, FormStateInterf + unset($element["widget"][0]['summary']['#attached']); ... + $element["widget"][0]['summary']['#required'] = TRUE; ... + if ($element["widget"][0]['summary']['#description'] == $default_summary_text) { + unset($element["widget"][0]['summary']['#description']); ... + $element["widget"][0]['summary']['#description'] = $summary_help_setting; ... + $element["widget"][0]['summary']['#attributes']['placeholder'] = [$summary_placeholder_setting];
Please don't mix single quotes and double quotes. ("widget")
- Status changed to Needs review
about 1 year ago 11:49am 16 October 2023 - ๐ซ๐ฎFinland sokru
I think we should rather use
hook_field_widget_single_element_form_alter
so it works also on multivalue fields and is much smaller change. - Status changed to RTBC
about 1 year ago 4:15pm 12 December 2023 - ๐ฌ๐งUnited Kingdom JennyDumitrescu
Tested #5 on our site as we were having issues with this module not working and it's added the description so working great for our use case.
- ๐ธ๐ฆSaudi Arabia martins.bruvelis Thuwal
martins.bruvelis โ made their first commit to this issueโs fork.
- Merge request !4Apply @sokru patch from issue 3385149 comment #5 that uses... โ (Merged) created by martins.bruvelis
- ๐บ๐ธUnited States duckydan
I still had to apply a patch to make this module work.
However, I needed to update the patch slightly for it to apply.
- Status changed to Needs review
4 months ago 1:22pm 7 August 2024 - Status changed to RTBC
4 months ago 2:32pm 7 August 2024 - ๐ซ๐ฎFinland sokru
Rebased the MR, hide the patch-files. Pipelines are green, so setting the status to RTBC.
- ๐ฆ๐บAustralia kim.pepper ๐โโ๏ธ๐ฆ๐บSydney, Australia
kim.pepper โ made their first commit to this issueโs fork.
-
kim.pepper โ
committed 9fdb5e13 on 2.x authored by
martins.bruvelis โ
Issue #3385149 by sokru, duckydan, sachin.bansal27, jwilson3:...
-
kim.pepper โ
committed 9fdb5e13 on 2.x authored by
martins.bruvelis โ
- Status changed to Fixed
4 months ago 10:16pm 7 August 2024 - ๐ฆ๐บAustralia kim.pepper ๐โโ๏ธ๐ฆ๐บSydney, Australia
Committed to 2.x. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.