We support now the indication of what field changed if no revision log is provided.
There is still no indication about the amount / severity of changes that have been applied.
As an idea to better indicate change:
Git displays on the terminal repeatingly + or - varying the amount of changes. Or sometimes % change. And a summary on a merge.
Example:
ccore/lib/Drupal/Core/Session/SessionManager.php | 3 ++-
.../src/Controller/TestPageTestController.php | 3 ++-
core/tests/Drupal/FunctionalTests/BrowserTestBaseTest.php | 15 +++++++++++++++
core/tests/Drupal/Tests/BrowserTestBase.php | 8 ++++++--
4 files changed, 25 insertions(+), 4 deletions(-)
So the challenge would be that we need to trigger the diff engine to know how many characters / words / lines have been removed and added.
Title +3/-15, Body +1k/-3.1k
Or even more simplified, we could output the change in length?
Title -34%, Body -15%
Title -12, Body -2.1k
Title 28>16, Body 14k>11.9k
But i guess those are lots of ideas for follow-ups... But not yet sure what we need to boost UX.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | Screenshot from 2016-09-19 12-10-49.png | 49.43 KB | johnchque |
| #2 | indicate_change_amount-2801635-2.patch | 1.46 KB | johnchque |
Comments
Comment #2
johnchqueMade some changes, Looks nice IMHO. The problem here is the performance, since it basically does the complete diff to lest the changes.
Comment #4
miro_dietikerHm, i don't like this text'ish representation of structural information.
I think we would need to show it line by line.
I created two issue to care about the line wrappings:
#2802417: Remove revision log in revision listing from Revision column
#2802813: Fix revision overview headings
The table will look much less compressed after fixing those.
But still, i don't know if we can do this right...
Another problem here is the hierarchical Paragraphs label.
I think we should do mocks first for this direction. Not a release priority for me, we have already changed fields indicated.