When you create a Field Type of Text (formatted, long, with summary) you get an option to check for Summary Input which allows you to have a field to explicitly set a Summary. This field is only visible, however, if the Allowed number of values in Field settings is Limited to 1.
Using the 'Unlimited' option or defining values greater than 1 causes the form to auto-hide the Summary field and label. The markup is there, but inline css on the element is setting the display to none.
<div class="js-text-summary-wrapper text-summary-wrapper" style="display: none;">
...
</div>
Comment | File | Size | Author |
---|---|---|---|
#11 | 2826356--before--patch--pic.png | 38.69 KB | vikashsoni |
#7 | 2826356-after_patch-4-2.png | 17.09 KB | Abhijith S |
#7 | 2826356_after_patch-4-1.png | 17.45 KB | Abhijith S |
#7 | 2826356-before_patch.png | 20.38 KB | Abhijith S |
#6 | Summary text visible_afterPatch.JPG | 55.25 KB | pankaj.singh |
Issue fork drupal-2826356
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
Revalis CreditAttribution: Revalis commentedI noticed I wasn't on the latest version, so updated to 8.2.2 and tried again. Same issue still exists.
Comment #3
shabana.navas CreditAttribution: shabana.navas at Acro Commerce commentedThe issue arises because the label for the unlimited textarea has a 'visually-hidden' class added to it. So, for fields with unlimited cardinality, instead of nesting the 'edit-link' underneath the label, we can put it before the label (see attached image).
Comment #4
grndlvl CreditAttribution: grndlvl at GollyGood Software commentedRe-roll against latest dev
Comment #6
pankaj.singh CreditAttribution: pankaj.singh as a volunteer and at Srijan | A Material+ Company for Drupal India Association commentedTested the patch on 8.9. Its working on my end.
Please find the SS attached for ref.
Comment #7
Abhijith S CreditAttribution: Abhijith S as a volunteer and at Zyxware Technologies commentedApplied patch #4 and it works .The summary section will be displayed in the form after the patch is applied. Including screenshots.
before:
after patch:
after patch(expanded summary):
Comment #8
Abhijith S CreditAttribution: Abhijith S as a volunteer and at Zyxware Technologies commentedComment #9
alexpottWe need to change text.es6.js and then use the build tools to generate text.js - see https://www.drupal.org/node/2815083
Also we should add a test for this.
Comment #11
vikashsoni CreditAttribution: vikashsoni as a volunteer and at Zyxware Technologies commentedI have checked this issue in drupal -9.3.x-dev
There is no issue exists summery field is coming properly when the textfield has unlimited value
for ref sharing screenshot....
Comment #12
luisrhaas CreditAttribution: luisrhaas at CI&T commentedComment #14
luisrhaas CreditAttribution: luisrhaas at CI&T commentedback to 9.2.x-dev
Comment #16
luisrhaas CreditAttribution: luisrhaas at CI&T commentedHi!
I opened a MR with patch #4 and made some tests using multiple text field with summary configuration visible/invisible.
Please let me know if need something else or it's ok.
Comment #17
nod_This should have been fixed by #2817081: Show 'Edit summary' links on multi-value long text field with summary. Can you confirm the bug happens in 9.3 or 9.4?
Comment #20
quietone CreditAttribution: quietone at PreviousNext commented@nod_, thanks.
I tested this on Drupal 10.1.x, umami install. I created a new content type with a multi-alue field of Text (formatted, long with summary). I checked the summary input box. When creating content I was able to add multiple body and summary values for the test field. This does appear to be fixed.
This is a duplicate of #2817081: Show 'Edit summary' links on multi-value long text field with summary which was created 1 month before this issue. It is a good reminder to search for duplicate issues.
I am moving credit to the other issue. No credit is given for duplicate screenshots.
Comment #21
quietone CreditAttribution: quietone at PreviousNext commented