Re: [Powertop] [PATCH] Remove unneded escaping backslashes from CSS

Dear Igor,


Am Freitag, den 14.09.2012, 13:46 +0400 schrieb Igor Zhbanov:
> Remove unneded escaping backslashes from CSS.

copying the summary into the body is not useful, because it adds no
further information.

Please also add why the escaping was added beforehand and why it is
unneeded.

> ---
>  src/powertop.css |   16 ++++++++--------
>  1 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/src/powertop.css b/src/powertop.css
> index 087e384..5d303dc 100644
> --- a/src/powertop.css
> +++ b/src/powertop.css
> @@ -21,14 +21,14 @@ $(document).ready(function(){
>  	$('#tuning h2').addClass('hide');
>  	$('#tuning table').addClass('hide');
>  
> -	$('#top').append('<div class=\"SystemButton\"   onclick=\"toggleSystem()\">System Info<\\/div>');
> -	$('#top').append('<div class=\"SummaryButton\"  onclick=\"toggleSummary()\">Summary<\\/div>');
> -	$('#top').append('<div class=\"CpuidleButton\"  onclick=\"toggleCpuidle()\">CPU Idle<\\/div>');
> -	$('#top').append('<div class=\"CpufreqButton\"  onclick=\"toggleCpufreq()\">CPU Frequency<\\/div>');
> -	$('#top').append('<div class=\"SoftwareButton\" onclick=\"toggleSoftware()\">Software info<\\/div>');
> -	$('#top').append('<div class=\"DeviceButton\"   onclick=\"toggleDevice()\">Device Info<\\/div>');
> -	$('#top').append('<div class=\"TuningButton\"   onclick=\"toggleTuning()\">Tuning<\\/div>');
> -	$('#top').append('<div class=\"AllButton\"      onclick=\"toggleAll()\">All<\\/div>');
> +	$('#top').append('<div class=\"SystemButton\"   onclick=\"toggleSystem()\">System Info</div>');
> +	$('#top').append('<div class=\"SummaryButton\"  onclick=\"toggleSummary()\">Summary</div>');
> +	$('#top').append('<div class=\"CpuidleButton\"  onclick=\"toggleCpuidle()\">CPU Idle</div>');
> +	$('#top').append('<div class=\"CpufreqButton\"  onclick=\"toggleCpufreq()\">CPU Frequency</div>');
> +	$('#top').append('<div class=\"SoftwareButton\" onclick=\"toggleSoftware()\">Software info</div>');
> +	$('#top').append('<div class=\"DeviceButton\"   onclick=\"toggleDevice()\">Device Info</div>');
> +	$('#top').append('<div class=\"TuningButton\"   onclick=\"toggleTuning()\">Tuning</div>');
> +	$('#top').append('<div class=\"AllButton\"      onclick=\"toggleAll()\">All</div>');
>  	$('#top .SummaryButton').toggleClass('pressed');
>  
>      }


Thanks,

Paul

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iEYEABECAAYFAlBTAMQACgkQPX1aK2wOHVhVRACeORVHh+EPi327dEWUGLvwZx4F
Gv8AnRrgoWU8ifjCS60QRRkrzA0Y6aTP
=jv0z
-----END PGP SIGNATURE-----