diff mbox series

[7/7] pakfire.cgi: Cosmetic fixes

Message ID 20220508120952.52-7-hofmann@leo-andres.de
State Accepted
Commit 2b5253bbbb3acb6f276040ffe095f7380ea3991d
Headers
Series [1/7] pakfire.cgi: Separate command processing and HTML generation |

Commit Message

Leo-Andres Hofmann May 8, 2022, 12:09 p.m. UTC 
  Add formatting to improve readability of dependencies list header.

Signed-off-by: Leo-Andres Hofmann <hofmann@leo-andres.de>
---
 html/cgi-bin/pakfire.cgi | 6 ++++--
 langs/fr/cgi-bin/fr.pl   | 2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)

Comments

Peter Müller May 8, 2022, 1:12 p.m. UTC | #1 
Acked-by: Peter Müller <peter.muelle@ipfire.org>

> Add formatting to improve readability of dependencies list header.
> 
> Signed-off-by: Leo-Andres Hofmann <hofmann@leo-andres.de>
> ---
>  html/cgi-bin/pakfire.cgi | 6 ++++--
>  langs/fr/cgi-bin/fr.pl   | 2 +-
>  2 files changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/html/cgi-bin/pakfire.cgi b/html/cgi-bin/pakfire.cgi
> index 489b07a6d..3e8dc5460 100644
> --- a/html/cgi-bin/pakfire.cgi
> +++ b/html/cgi-bin/pakfire.cgi
> @@ -283,7 +283,8 @@ if (($cgiparams{'ACTION'} eq 'install') && ($pagemode eq $PM_DEFAULT)) {
>  	my @pkgs = split(/\|/, $cgiparams{'INSPAKS'});
>  	my @output = &General::system_output("/usr/local/bin/pakfire", "resolvedeps", "--no-colors", @pkgs);
>  	print <<END;
> -	<table><tr><td colspan='2'>$Lang::tr{'pakfire install package'} @pkgs $Lang::tr{'pakfire possible dependency'}
> +	<table style="width: 100%"><tr><td colspan='2'><p>$Lang::tr{'pakfire install package'} <strong>@{pkgs}</strong><br>
> +		$Lang::tr{'pakfire possible dependency'}</p>
>  		<pre>
>  END
>  	foreach (@output) {
> @@ -321,7 +322,8 @@ END
>  	my @pkgs = split(/\|/, $cgiparams{'DELPAKS'});
>  	my @output = &General::system_output("/usr/local/bin/pakfire", "resolvedeps", "--no-colors", @pkgs);
>  	print <<END;
> -	<table><tr><td colspan='2'>$Lang::tr{'pakfire uninstall package'} @pkgs $Lang::tr{'pakfire possible dependency'}
> +	<table style="width: 100%"><tr><td colspan='2'><p>$Lang::tr{'pakfire uninstall package'} <strong>@{pkgs}</strong><br>
> +		$Lang::tr{'pakfire possible dependency'}</p>
>  		<pre>
>  END
>  	foreach (@output) {
> diff --git a/langs/fr/cgi-bin/fr.pl b/langs/fr/cgi-bin/fr.pl
> index bd17df4ee..6dbeebc16 100644
> --- a/langs/fr/cgi-bin/fr.pl
> +++ b/langs/fr/cgi-bin/fr.pl
> @@ -2055,7 +2055,7 @@
>  'pakfire last package update' => 'Dernière mise à jour de la liste des paquets : ',
>  'pakfire last serverlist update' => 'Dernière mise à jour de la liste des serveurs : ',
>  'pakfire last update' => 'Dernière mise à jour : ',
> -'pakfire possible dependency' => '<br><br>Il y a peut-être des dépendances, voici la liste des paquets qu\'il faut (dés)installer.<br>',
> +'pakfire possible dependency' => 'Il y a peut-être des dépendances, voici la liste des paquets qu\'il faut (dés)installer.',
>  'pakfire register' => 'S\'inscrire au serveur pakfire :',
>  'pakfire return' => 'Retour à Pakfire',
>  'pakfire system state' => 'Statut système PakFire ',
diff mbox series

Patch

diff --git a/html/cgi-bin/pakfire.cgi b/html/cgi-bin/pakfire.cgi
index 489b07a6d..3e8dc5460 100644
--- a/html/cgi-bin/pakfire.cgi
+++ b/html/cgi-bin/pakfire.cgi
@@ -283,7 +283,8 @@  if (($cgiparams{'ACTION'} eq 'install') && ($pagemode eq $PM_DEFAULT)) {
 	my @pkgs = split(/\|/, $cgiparams{'INSPAKS'});
 	my @output = &General::system_output("/usr/local/bin/pakfire", "resolvedeps", "--no-colors", @pkgs);
 	print <<END;
-	<table><tr><td colspan='2'>$Lang::tr{'pakfire install package'} @pkgs $Lang::tr{'pakfire possible dependency'}
+	<table style="width: 100%"><tr><td colspan='2'><p>$Lang::tr{'pakfire install package'} <strong>@{pkgs}</strong><br>
+		$Lang::tr{'pakfire possible dependency'}</p>
 		<pre>
 END
 	foreach (@output) {
@@ -321,7 +322,8 @@  END
 	my @pkgs = split(/\|/, $cgiparams{'DELPAKS'});
 	my @output = &General::system_output("/usr/local/bin/pakfire", "resolvedeps", "--no-colors", @pkgs);
 	print <<END;
-	<table><tr><td colspan='2'>$Lang::tr{'pakfire uninstall package'} @pkgs $Lang::tr{'pakfire possible dependency'}
+	<table style="width: 100%"><tr><td colspan='2'><p>$Lang::tr{'pakfire uninstall package'} <strong>@{pkgs}</strong><br>
+		$Lang::tr{'pakfire possible dependency'}</p>
 		<pre>
 END
 	foreach (@output) {
diff --git a/langs/fr/cgi-bin/fr.pl b/langs/fr/cgi-bin/fr.pl
index bd17df4ee..6dbeebc16 100644
--- a/langs/fr/cgi-bin/fr.pl
+++ b/langs/fr/cgi-bin/fr.pl
@@ -2055,7 +2055,7 @@ 
 'pakfire last package update' => 'Dernière mise à jour de la liste des paquets : ',
 'pakfire last serverlist update' => 'Dernière mise à jour de la liste des serveurs : ',
 'pakfire last update' => 'Dernière mise à jour : ',
-'pakfire possible dependency' => '<br><br>Il y a peut-être des dépendances, voici la liste des paquets qu\'il faut (dés)installer.<br>',
+'pakfire possible dependency' => 'Il y a peut-être des dépendances, voici la liste des paquets qu\'il faut (dés)installer.',
 'pakfire register' => 'S\'inscrire au serveur pakfire :',
 'pakfire return' => 'Retour à Pakfire',
 'pakfire system state' => 'Statut système PakFire ',