[Libreoffice] [PATCH] Re: UI Libre Office used 'PI'

Hi Michael, *,

On Tue, 17 May 2011 12:26:09 +0100, Michael Meeks <michael.meeks@novell.com> wrote:
> On Wed, 2011-05-11 at 08:31 -0700, ape wrote:
> > UI Libre Office used 'pi'... The unit of measure 'pica' is the abbreviated
> > name 'pi'. This is not compliant with CSS1 ('pica' = 'pc'). Why?
>
>       No idea - where is it used ? is it a space issue ? do you have a tested
> patch to the relevant .src file ? :-) If you do we can review / merge
> it.
Please review the attached one.
I have observed the behavior entering "1pi"/"1pc" at, e.g., a field
in a dialog of Calc ("Format Cells" -> "Borders" -> "Spacing to contents").

Cheers,
-- Takeshi Abe

From c25f4201bc57c7596f7292fe25a3c12b5acbe164 Mon Sep 17 00:00:00 2001
From: Takeshi Abe <tabe@fixedpoint.jp>
Date: Wed, 18 May 2011 20:09:24 +0900
Subject: [PATCH] abbreviation "pc" for pica, instead of "pi"

according to http://www.w3.org/TR/CSS2/syndata.html#length-units
---
 vcl/source/src/units.src |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/vcl/source/src/units.src b/vcl/source/src/units.src
index aff92ad..9deb287 100644
--- a/vcl/source/src/units.src
+++ b/vcl/source/src/units.src
@@ -42,7 +42,7 @@ StringArray SV_FUNIT_STRINGS
         < "twips" ; FUNIT_TWIP ; > ;
         < "twip" ; FUNIT_TWIP ; > ;
         < "pt" ; FUNIT_POINT ; > ;
-        < "pi" ; FUNIT_PICA ; > ;
+        < "pc" ; FUNIT_PICA ; > ;
         < "\"" ; FUNIT_INCH ; > ;
         < "in" ; FUNIT_INCH ; > ;
         < "inch" ; FUNIT_INCH ; > ;
-- 
1.7.2.5