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.
Problem/Motivation
When there are multivalue fields that have changed, the summary might show the same label.
Proposed resolution
Us the delta for making the summary nicer.
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#8 | interdiff-2824642-6-8.txt | 982 bytes | johnchque |
#8 | use_deltas_for-2824642-8.patch | 3.02 KB | johnchque |
#6 | Screenshot from 2016-11-04 09-49-02.png | 48.73 KB | johnchque |
#6 | use_deltas_for-2824642-6.patch | 2.49 KB | johnchque |
#5 | Screenshot from 2016-11-04 09-44-28.png | 45.65 KB | johnchque |
Comments
Comment #2
johnchqueDidn't know that the patch was so small. This might break some tests.
Comment #3
miro_dietikerFor end users, number counts start at 1, so you would need to do +1 for number display.
Indexes are typically in square brackets such as:
Paragraphs [1] > Text
Math would put the index into lower, such as
<sub>
tag.Paragraphs ₁
^ I think this is best. :-)
If not, we could try to remove that bracket noise completely:
Paragraphs > 1 > Text
Paragraphs - 1 > Text
Or even remove the Arrow noise too?
Paragraphs - 1 - Text
Or we could look at some extremely slim unicode arrows, just some quick examples, there are more:
➢ ⤇ ⦊ ⭢ ⮞
Comment #4
johnchqueTried with sub tag, looks good IMHO. :) Changed a bit the tags allowed.
Comment #5
johnchqueNow it starts with one. It compares the correct entities. Removed the arrow. Wondering if we should separate the fields with dash instead of comma?
Comment #6
johnchqueImploding with dash instead. ;)
Comment #8
johnchqueAs discussed using dash for separating fields. Fixed tests and improved the summary method a bit.
Comment #10
miro_dietikerOK Cool, committing.
Discussed follow-up: we should remove the index if cardinality 1.
And i still feel like the " - " can be more compact and make things more clearly belong together if we use a UTF-8 arrow and even remove the spaces...
Comment #11
johnchque#2824964: Do not display deltas when field cardinality is 1 Created follow-up.