From d6d803ba02ac62c49454e459de46de94d5932e56 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= <ppisar@redhat.com>
Date: Thu, 9 Jan 2025 17:20:42 +0100
Subject: [PATCH] I18N: Annotate indentation of the transaction summary
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Translators are recommend to align the messages to prevent from
a broken table:

    Podsumowanie transakcji:
     Instalowanie:         6 pakietów
     Aktualizowanie:         33 pakietów
     Zastępowanie:         48 pakietów
     Usuwanie:         199 pakietów

(Last arguments moved to a new line to follow new clang-format.)

Relates: https://bugzilla.redhat.com/show_bug.cgi?id=2336561
---
 libdnf5-cli/output/transaction_table.cpp | 29 ++++++++++++++++++------
 1 file changed, 22 insertions(+), 7 deletions(-)

diff --git a/libdnf5-cli/output/transaction_table.cpp b/libdnf5-cli/output/transaction_table.cpp
index 5ed82df86..f1da90a8a 100644
--- a/libdnf5-cli/output/transaction_table.cpp
+++ b/libdnf5-cli/output/transaction_table.cpp
@@ -128,48 +128,61 @@ class TransactionSummary {
         if (installs != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
-                    P_(" Installing:      {:4} package\n", " Installing:      {:4} packages\n", installs), installs)
+                    // Keep the replaceble number aligned across all messages.
+                    P_(" Installing:      {:4} package\n", " Installing:      {:4} packages\n", installs),
+                    installs)
                     .c_str(),
                 fd);
         }
         if (reinstalls != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
-                    P_(" Reinstalling:    {:4} package\n", " Reinstalling:    {:4} packages\n", reinstalls), reinstalls)
+                    // Keep the replaceble number aligned across all messages.
+                    P_(" Reinstalling:    {:4} package\n", " Reinstalling:    {:4} packages\n", reinstalls),
+                    reinstalls)
                     .c_str(),
                 fd);
         }
         if (upgrades != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
-                    P_(" Upgrading:       {:4} package\n", " Upgrading:       {:4} packages\n", upgrades), upgrades)
+                    // Keep the replaceble number aligned across all messages.
+                    P_(" Upgrading:       {:4} package\n", " Upgrading:       {:4} packages\n", upgrades),
+                    upgrades)
                     .c_str(),
                 fd);
         }
         if (replaced != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
-                    P_(" Replacing:       {:4} package\n", " Replacing:       {:4} packages\n", replaced), replaced)
+                    // Keep the replaceble number aligned across all messages.
+                    P_(" Replacing:       {:4} package\n", " Replacing:       {:4} packages\n", replaced),
+                    replaced)
                     .c_str(),
                 fd);
         }
         if (removes != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
-                    P_(" Removing:        {:4} package\n", " Removing:        {:4} packages\n", removes), removes)
+                    // Keep the replaceble number aligned across all messages.
+                    P_(" Removing:        {:4} package\n", " Removing:        {:4} packages\n", removes),
+                    removes)
                     .c_str(),
                 fd);
         }
         if (downgrades != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
-                    P_(" Downgrading:     {:4} package\n", " Downgrading:     {:4} packages\n", downgrades), downgrades)
+                    // Keep the replaceble number aligned across all messages.
+                    P_(" Downgrading:     {:4} package\n", " Downgrading:     {:4} packages\n", downgrades),
+                    downgrades)
                     .c_str(),
                 fd);
         }
         if (reason_changes != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
+                    // Keep the replaceble number aligned across all messages.
                     P_(" Changing reason: {:4} package\n", " Changing reason: {:4} packages\n", reason_changes),
                     reason_changes)
                     .c_str(),
@@ -178,7 +191,9 @@ class TransactionSummary {
         if (skips != 0) {
             std::fputs(
                 libdnf5::utils::sformat(
-                    P_(" Skipping:        {:4} package\n", " Skipping:        {:4} packages\n", skips), skips)
+                    // Keep the replaceble number aligned across all messages.
+                    P_(" Skipping:        {:4} package\n", " Skipping:        {:4} packages\n", skips),
+                    skips)
                     .c_str(),
                 fd);
         }