Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Hi,
Could we somehow allow the editor to chose what would be the header? I would like to have an option to set the first row or first column as header and have a correct output.
| Comment | File | Size | Author |
|---|---|---|---|
| #16 | 2848794-row-headers-16.patch | 2.54 KB | Lukas von Blarer |
| #14 | 2848794-row-headers-14.patch | 2.54 KB | Lukas von Blarer |
| #12 | 2848794-row-headers-12.patch | 2.62 KB | Lukas von Blarer |
| #3 | new_field_formatter.patch | 1.44 KB | AstonVictor |
| #2 | 77b98e3db3.png | 8.14 KB | AstonVictor |
Comments
Comment #2
AstonVictor CreditAttribution: AstonVictor at DevBranch commentedHi there.
This patch added new field formatter - Tabular View (column).
Also, see the screenshot.
Comment #3
AstonVictor CreditAttribution: AstonVictor at DevBranch commentedComment #4
Lukas von BlarerWe should also support column and row headers at the same time.
Comment #5
lolandese CreditAttribution: lolandese as a volunteer commentedComment #7
lolandese CreditAttribution: lolandese as a volunteer commentedComment #8
lolandese CreditAttribution: lolandese as a volunteer commentedSeems Code Sniffer detects an unused `use` statement.
Comment #9
Lukas von BlarerWe should provide these options in the default field formatter as settings. What I don't understand is the current solution of having the setting of table headers in the field type instead of the formatter. Does this make sense?
Comment #10
lolandese CreditAttribution: lolandese as a volunteer commented@Lukas #9:
Yes. For D7 this was actually fixed and should be done also for D8. See #2841782: Move display related settings from manage fields to manage display.
Comment #11
Lukas von BlarerI created an issue for this: #3019613: Move display related settings from manage fields to manage display and add options
Comment #12
Lukas von BlarerHere we go. I Had a little fight with render arrays, but now it works.
This depends on that patch: https://www.drupal.org/project/tablefield/issues/3019613#comment-12889642
Comment #14
Lukas von BlarerRe-rolling the patch against #3019613: Move display related settings from manage fields to manage display and add options and adding CS fixes.
Comment #16
Lukas von BlarerRe-rolling again.
Comment #18
lolandese CreditAttribution: lolandese as a volunteer commentedThis seems to be fixed by #3019613: Move display related settings from manage fields to manage display and add options. Please check and close as duplicate if this is the case.
Comment #19
lolandese CreditAttribution: lolandese as a volunteer commentedSolved in #3019613: Move display related settings from manage fields to manage display and add options.