[Libreoffice-commits] core.git: vcl/aqua vcl/inc

[Boris Dušek]

 vcl/aqua/source/a11y/aqua11ytextattributeswrapper.mm |   20 +++++++++++++++----
 vcl/aqua/source/a11y/aqua11ywrapper.mm               |    9 ++++++++
 vcl/inc/aqua/aqua11ywrapper.h                        |    3 ++
 3 files changed, 28 insertions(+), 4 deletions(-)

New commits:
commit 3e9ae4da947cd9c3f75b6e4942f2ba52337df075
Author: Boris Dušek <me at dusek.me>
Date:   Sun Aug 11 08:44:26 2013 +0200

    fdo#67957: Font name reported in AXFont always Times New Roman
    
    This is a partial fix. Now VoiceOver does report changes in font faces
    but it works correctly only when the paragraph text style has font
    "Times New Roman". If it has not, then parts of the text with
    "Times New Roman" have not change in font reported, but parts with
    font different both from the paragraph style font and "Times New Roman"
    do have font change reported.
    
    In other words, the default font for paragraph is still "Times New Roman"
    in accessibility even though sometimes it's not true.
    
    This also fixes font size being reported only when bold or italic is set,
    and has more robustness for handling mixed bold/italic when at least one
    of them is set in the paragraph style as well.
    
    Change-Id: Id0715727d04cd9b814aa0e4093939cd3e6abe897
    Reviewed-on: https://gerrit.libreoffice.org/5344
    Reviewed-by: Tor Lillqvist <tml at iki.fi>
    Tested-by: Tor Lillqvist <tml at iki.fi>

diff --git a/vcl/aqua/source/a11y/aqua11ytextattributeswrapper.mm b/vcl/aqua/source/a11y/aqua11ytextattributeswrapper.mm
index 2e4ab40..e8a676e 100644
--- a/vcl/aqua/source/a11y/aqua11ytextattributeswrapper.mm
+++ b/vcl/aqua/source/a11y/aqua11ytextattributeswrapper.mm
@@ -50,7 +50,7 @@ using namespace ::rtl;
 }
 
 +(int)convertBoldStyle:(PropertyValue)property {
-    int boldStyle = 0;
+    int boldStyle = NSUnboldFontMask;
     float value = 0;
     property.Value >>= value;
     if ( value == ::css_awt::FontWeight::SEMIBOLD
@@ -63,7 +63,7 @@ using namespace ::rtl;
 }
 
 +(int)convertItalicStyle:(PropertyValue)property {
-    int italicStyle = 0;
+    int italicStyle = NSUnitalicFontMask;
     sal_Int16 value = property.Value.get< ::css_awt::FontSlant>();
     if ( value == ::css_awt::FontSlant_ITALIC ) {
         italicStyle = NSItalicFontMask;
@@ -198,10 +198,22 @@ using namespace ::rtl;
     if ( wrapperStore != nil ) { // default
         [ wrapperStore setDefaultFontname: CreateNSString ( fontname ) ];
         [ wrapperStore setDefaultFontsize: fontsize ];
+        [ wrapperStore setDefaultFonttraits: fonttraits ];
         NSFont * font = [ [ NSFontManager sharedFontManager ] fontWithFamily: CreateNSString ( fontname ) traits: fonttraits weight: 0 size: fontsize ];
         [ AquaA11yTextAttributesWrapper addFont: font toString: string forRange: range ];
-    } else if ( wrapper != nil && fonttraits != 0 ) { // attribute run and bold and/or italic was found
-        NSFont * font = [ [ NSFontManager sharedFontManager ] fontWithFamily: [ wrapper defaultFontname ] traits: fonttraits weight: 0 size: [ wrapper defaultFontsize ] ];
+    } else if ( wrapper != nil) { // attribute run and bold and/or italic was found
+        NSString *fontName = nil;
+        if (fontname.isEmpty())
+            fontName = [wrapper defaultFontname];
+        else
+            fontName = CreateNSString(fontname);
+        if (!(fonttraits & (NSBoldFontMask | NSUnboldFontMask)))
+            fonttraits |= [wrapper defaultFonttraits] & (NSBoldFontMask | NSUnboldFontMask);
+        if (!(fonttraits & (NSItalicFontMask | NSUnitalicFontMask)))
+            fonttraits |= [wrapper defaultFonttraits] & (NSItalicFontMask | NSUnitalicFontMask);
+        if (fontsize == 0.0)
+            fontsize = [wrapper defaultFontsize];
+        NSFont * font = [ [ NSFontManager sharedFontManager ] fontWithFamily: fontName traits: fonttraits weight: 0 size: fontsize ];
         [ AquaA11yTextAttributesWrapper addFont: font toString: string forRange: range ];
     }
     [ pool release ];
diff --git a/vcl/aqua/source/a11y/aqua11ywrapper.mm b/vcl/aqua/source/a11y/aqua11ywrapper.mm
index bb0d81b..375997f 100644
--- a/vcl/aqua/source/a11y/aqua11ywrapper.mm
+++ b/vcl/aqua/source/a11y/aqua11ywrapper.mm
@@ -82,6 +82,7 @@ static std::ostream &operator<<(std::ostream &s, NSPoint point) {
 
 -(void) setDefaults: (Reference < XAccessibleContext >) rxAccessibleContext {
     mDefaultFontsize = 0.0;
+    mDefaultFonttraits = 0;
     mpDefaultFontname = nil;
     mpReferenceWrapper = new ReferenceWrapper;
     mActsAsRadioGroup = NO;
@@ -1145,6 +1146,14 @@ Reference < XAccessibleContext > hitTestRunner ( com::sun::star::awt::Point poin
     return mDefaultFontsize;
 }
 
+-(void)setDefaultFonttraits:(int)fonttraits {
+    mDefaultFonttraits = fonttraits;
+}
+
+-(int)defaultFonttraits {
+    return mDefaultFonttraits;
+}
+
 -(void)setActsAsRadioGroup:(BOOL)actsAsRadioGroup {
     mActsAsRadioGroup = actsAsRadioGroup;
 }
diff --git a/vcl/inc/aqua/aqua11ywrapper.h b/vcl/inc/aqua/aqua11ywrapper.h
index da34264..185536a 100644
--- a/vcl/inc/aqua/aqua11ywrapper.h
+++ b/vcl/inc/aqua/aqua11ywrapper.h
@@ -54,6 +54,7 @@ struct ReferenceWrapper
     ReferenceWrapper * mpReferenceWrapper;
     NSString * mpDefaultFontname;
     float mDefaultFontsize;
+    int mDefaultFonttraits;
     BOOL mActsAsRadioGroup;
     BOOL mIsTableCell;
 }
@@ -94,6 +95,8 @@ struct ReferenceWrapper
 -(NSString *)defaultFontname;
 -(void)setDefaultFontsize:(float)fontsize;
 -(float)defaultFontsize;
+-(void)setDefaultFonttraits:(int)fonttraits;
+-(int)defaultFonttraits;
 +(void)setPopupMenuOpen:(BOOL)popupMenuOpen;
 -(::com::sun::star::accessibility::XAccessibleAction *)accessibleAction;
 -(::com::sun::star::accessibility::XAccessibleContext *)accessibleContext;