Revert back is redundant

Seems like a no brainer to me.

http://www.vocabulary.com/dictionary/revert

But...

    drupal_set_message(t('@type %title has been reverted back to the revision from %revision-date.', array('@type' => node_get_type_label($this->revision), '%title' => $this->revision->label(), '%revision-date' => format_date($original_revision_timestamp))));

in \Drupal\node\Form\NodeRevisionDeleteForm

You will also need to fix \Drupal\node\Tests\NodeRevisionsAllTest and \Drupal\node\Tests\NodeRevisionsTest

I'll have a look at the tests.

Patch applies cleanly and also I didn't find any other places in core containing this redundancy.

Retitling so that the issue title fits the patch. Also redundancy is not a bug.

1. +++ b/core/modules/quickedit/js/views/EditorView.js
   @@ -163,7 +163,7 @@
   -     * Reverts the modified value back to the original value (before editing
   +     * Reverts the modified value to the original value (before editing
         * started).
   

   Let's change this to * Reverts the modified value to the original value, before editing started. so it fits our standard.

2. +++ b/core/modules/views_ui/src/Tests/OverrideDisplaysTest.php
   @@ -182,7 +182,7 @@ function testRevertAllDisplays() {
   -    // Revert the title of the block back to the default ones, but submit some
   +    // Revert the title of the block to the default ones, but submit some
        // new values to be sure that the new value is not stored.
   

   Comment should be reflowed.

1. Done.
2. Can you please clarify what needs to be done here? The comment fits the coding standards and is 74 chars wide.

@Ajit re point 2 new from the following line can be moved to the line that is being changed.

+++ b/core/modules/quickedit/js/views/EditorView.js
@@ -163,8 +163,8 @@
-     * Reverts the modified value to the original value (before editing
-     * started).
+     * Reverts the modified value to the original value, before editing
+     * started.

I think started, will now fit on the first line.

Thank you @Alex!
I have moved new from to the line that is being changed.
Couldn't move started above because the line length would then become 81.

+++ b/core/modules/quickedit/js/views/EditorView.js
@@ -163,8 +163,8 @@
+     * Reverts the modified value to the original value, before editing
+     * started.

We could reduce "original value" to "original" only to make it fit to one line.

But from the previous discussion, all is properly implemented.

This issue is a normal task so we need to outline how it fits within the allowable Drupal 8 beta criteria. Can someone add Drupal 8 beta phase evaluation template to the issue summary. It's allowable because we're not in string freeze. Also it would be great if the patch can be rerolled to have @miro_dietiker's suggested fix in #15 in.

Implementing the suggestion from #15.
I am not sure if I should set the status of the issue to needs review before updating the issue summary. So leaving the status as is.

Because this patch applies cleanly, implements all the suggested changes and translated strings are unfrozen in beta I will set this as RTBC .

Committed 8acb057 and pushed to 8.0.x. Thanks!

Thanks for adding the beta evaluation to the issue summary.

First comment needs a dot in the end.

#27 suggestion incorporated.

I think some of these could be considered end-user-facing strings (depends on the site; they are kind of in between) so based on https://www.drupal.org/node/1527558 we probably shouldn't be changing them and breaking translations in the middle of a stable release, given that this is a minor issue and not really a bug.

If it's a purely admin-facing string though, or certainly any examples that occur in code comments rather than the user interface, those we could still fix :)

Based on comment #30 I think we will mark this issue as won't fix.

Removing Novice task, issue currently too long for novice.

Tentatively marking this as won't fix based on #30 -- feel free to reopen if you disagree :)