Consistent tense for undo commands

qgis · Oct 6, 2017 · 3c87fe1 · 3c87fe1
1 parent 48c0fb3
commit 3c87fe1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/core/layout/qgslayoutitem.cpp b/src/core/layout/qgslayoutitem.cpp
@@ -100,7 +100,7 @@ void QgsLayoutItem::setId( const QString &id )
   }
 
   if ( !shouldBlockUndoCommands() )
-    mLayout->undoStack()->beginCommand( this, tr( "Change item ID" ) );
+    mLayout->undoStack()->beginCommand( this, tr( "Item ID changed" ) );
 
   mId = id;
 

diff --git a/src/core/layout/qgslayoutpagecollection.cpp b/src/core/layout/qgslayoutpagecollection.cpp
@@ -277,7 +277,7 @@ QList<int> QgsLayoutPageCollection::visiblePageNumbers( QRectF region ) const
 void QgsLayoutPageCollection::addPage( QgsLayoutItemPage *page )
 {
   if ( !mBlockUndoCommands )
-    mLayout->undoStack()->beginCommand( this, tr( "Add page" ) );
+    mLayout->undoStack()->beginCommand( this, tr( "Added page" ) );
   mPages.append( page );
   mLayout->addItem( page );
   reflow();
@@ -288,7 +288,7 @@ void QgsLayoutPageCollection::addPage( QgsLayoutItemPage *page )
 void QgsLayoutPageCollection::insertPage( QgsLayoutItemPage *page, int beforePage )
 {
   if ( !mBlockUndoCommands )
-    mLayout->undoStack()->beginCommand( this, tr( "Add page" ) );
+    mLayout->undoStack()->beginCommand( this, tr( "Added page" ) );
 
   if ( beforePage < 0 )
     beforePage = 0;
@@ -314,8 +314,8 @@ void QgsLayoutPageCollection::deletePage( int pageNumber )
 
   if ( !mBlockUndoCommands )
   {
-    mLayout->undoStack()->beginMacro( tr( "Remove page" ) );
-    mLayout->undoStack()->beginCommand( this, tr( "Remove page" ) );
+    mLayout->undoStack()->beginMacro( tr( "Removed page" ) );
+    mLayout->undoStack()->beginCommand( this, tr( "Removed page" ) );
   }
   emit pageAboutToBeRemoved( pageNumber );
   QgsLayoutItemPage *page = mPages.takeAt( pageNumber );