Bug 530085

Detail

Language: Oriya (or_IN)
Font: lohit-oriya-fonts-2.4.3-2.fc12.noarch
fontconfig-2.7.3-1.fc12.x86_64
Fedora: Rawhide (20Oct2009)
Installation: Fresh installation

here we dont have a scaling rules, like we have for meera fonts
so dont looks like fontconfig bug,

i guess there is some problem in fedora default theme only
reassigning to gnome-theme

while testing this bug, after login in or_IN locale, bug appears with Kwrite and firefox application

if we set window title font Lohit-Hindi, then everything works fine

This bug appears to have been reported against 'rawhide' during the Fedora 12 development cycle.
Changing version to '12'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

while testing same for other locale i found problem there as well

I faced the same problem while testing for Oriya interface (or_IN) in most of the applications in Rawhide.

i tested with changing 'hhea' true type fonts table in lohit fonts.
but this is different problem

whenever we login in any indic locale or sinhala says, things work well as far
windows title bar text is in local script.

but as soon as we open any application having titlebar in ascii, title bar
border gets displaced and its make appearances bad.

just border line placements keeps on varying as per default local script fonts
and ascii script fonts

Created attachment 401665 [details]
taken evince screenshot in kannada locale

if we open any other application having titlebar test in current locale script, it show titlebar proper without any problem of border misplacement

filed upstream bug against GTK+

https://bugzilla.gnome.org/show_bug.cgi?id=613778

I guess bug 574354 is duplicated.

Fedora theme is ClearLooks and some other themes work fine.
Probably I think downing the line position is better than narrowing the title height.
The following is the workaround but still needs the better fix.

--- /usr/share/themes/Clearlooks/metacity-1/metacity-theme-1.xml.orig	2010-03-25 22:15:04.000000000 +0900
+++ /usr/share/themes/Clearlooks/metacity-1/metacity-theme-1.xml	2010-03-26 18:39:30.000000000 +0900
@@ -223,7 +223,7 @@
 		<color value="shade/gtk:bg[SELECTED]/1.02"/>
 	</gradient>
 
-	<line color="shade/gtk:bg[SELECTED]/0.7" x1="1" y1="title_height + 6" x2="width - 2" y2="title_height + 6"/>
+	<line color="shade/gtk:bg[SELECTED]/0.7" x1="1" y1="title_height + 10" x2="width - 2" y2="title_height + 10"/>
 
 	<!-- ** border outline ** -->
 	<line color="shade/gtk:bg[NORMAL]/0.45" x1="0" y1="title_height + 6" x2="0" y2="height"/>
@@ -290,7 +290,7 @@
 		<color value="shade/gtk:bg[NORMAL]/0.95"/>
 	</gradient>
 
-	<line color="shade/gtk:bg[NORMAL]/0.65" x1="1" y1="title_height + 6" x2="width - 2" y2="title_height + 6"/>
+	<line color="shade/gtk:bg[NORMAL]/0.65" x1="1" y1="title_height + 10" x2="width - 2" y2="title_height + 10"/>
 	<rectangle color="shade/gtk:bg[NORMAL]/0.55" filled="false" x="0" y="0" width="width - 1" height="height - 1"/>
 </draw_ops>

Created attachment 403248 [details]
Patch for gnome-themes Clearlooks/metacity-theme-1.xml

If my suggestion is ok, I'll file an upstream bug later.

fill_env() assigns title_height and top_height in metacity ui/theme.c

fill_env ()
{
      env->top_height = info->fgeom->top_height;
  env->title_height = info->title_layout_height;
}

meta_frame_style_draw_with_style() calculates the font height and assigns the value in title_height.

meta_frame_style_draw_with_style ()
{
    pango_layout_get_pixel_extents (title_layout,
                                    NULL, &extents);

  draw_info.title_layout_height = title_layout ? extents.height : 0;
}

But meta_frame_layout_get_borders() calculates top_height with text_height + title_border.top + title_border.bottom so I think using top_height makes sense for the title vertical line.

meta_frame_layout_get_borders ()
{
  buttons_height = layout->button_height +
    layout->button_border.top + layout->button_border.bottom;
  title_height = text_height +
    layout->title_vertical_pad +
    layout->title_border.top + layout->title_border.bottom;

  if (top_height)
    {
      *top_height = MAX (buttons_height, title_height);
    }
}

fill_env() doesn't use info->fgeom->bottom_titlebar_edge so my suggestion is to use "top_height - 3".

(In reply to comment #10)
> But meta_frame_layout_get_borders() calculates top_height with text_height +
> title_border.top + title_border.bottom so I think using top_height makes sense
> for the title vertical line.

s/vertical line/horizontal line/

hi
i tried patch in comment #10 in fedora 13 but still no positive result

i followed following stpes:

1) apply patch to /usr/share/themes/Clearlooks/metacity-1/metacity-theme-1.xml
2) logoff and login in kn_IN (Kannada) locale
3) open any application having titlebar in latin/ascii 
4) open gnome-terminal or evince
5) Maximize windows you can see underline again in titlebar

looks like we need some fix from GTK or metacity itself

Thanks for the quick test.

(In reply to comment #12)
> 5) Maximize windows you can see underline again in titlebar

Ah, I forgot to modify the maximize property.
The value is also saved in the same metacity-theme-1.xml

I'll update it tomorrow.

Created attachment 403267 [details]
Patch for gnome-themes Clearlooks/metacity-theme-1.xml

(In reply to comment #13)
> I'll update it tomorrow.    

Revised the patch.

fujiwara thats great!!
It works fine now, i also tested for other locale things look fine 
so it is not creating any regression as well

it is also fixing long standing https://bugzilla.redhat.com/show_bug.cgi?id=523454


Matthias can you take a look at it?

Created attachment 403428 [details]
Patch for gnome-themes Clearlooks/metacity-theme-1.xml

Also applied the similar fix to ClearlooksClassic, Glossy and Inverted metacity themes.

It seems today gnome-themes was updated in rawhide. I'll check it later.

https://bugzilla.gnome.org/show_bug.cgi?id=614343

Fedora 13, with gnome-2-30 (gnome-themes-2.30.0-2.fc13.noarch), Punjabi (pa_IN)
locale is effected.

gnome-themes-2.28.1-3.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/gnome-themes-2.28.1-3.fc12

gnome-themes-2.28.1-3.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update gnome-themes'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/gnome-themes-2.28.1-3.fc12

did we missed pushing this update for F13?
i have tested same for F13 and its working fine, so i can quickly add 1 karma for it :)

gnome-themes-2.28.1-3.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.