Bug 48927 – Some links do not work on WebKit GTK+, and other links are displayed on top of the failed links.

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Ronald Clifford Buckman]

Created attachment 72834 [details]
nba.com on WebKitGTK+ 1.2.5

The links on the linkbar which has "NEWS, SCORES & SCHEDULES, VIDEO, ..." do not click in WebKit GTK+ 1.2.5.  Text links display over the linkbar.  W3.org validator did report 204 errors on the home page on November 3, 2010.

[Ronald Clifford Buckman]

Created attachment 74579 [details]
nba.com in Konqueror on Qt Webkit 4.7

The same display problem is in KHTML.  Likely due to HTML errors on the web page.

[Ronald Clifford Buckman]

Created attachment 76447 [details]
nba.com in rekonq and Arora on Qt WebKit 4.7

The bug is seen in Konqueror-WebKit and Rekonq, but not in Arora even though they all use the same layout engine.

[Ronald Clifford Buckman]

Created attachment 76449 [details]
nba.com in rekonq and Konqueror-KHTML,

This bug shows up in rekonq on Qt WebKit 4.7.  It no longer show on KHTML in Konqueror 4.5.  Also, the NBA score applet is not shown in rekonq or other Qt WebKit 4.7 browsers due to bug 50533.

[Martin Robinson]

*** Bug 51251 has been marked as a duplicate of this bug. ***

[Martin Robinson]

The issue here seems to the label element next to the search box and button. The label appears to exist only for accessibility and has has visilbity:hidden and font-size:0. WebKitGTK+ sets the default font size to 5 points by default, which is enough to bump the other text over. This doesn't break in Safari and Chrome, because Safari uses a smaller minimum font size and Chromium doesn't have one (as far as I know).

[Ronald Clifford Buckman]

The WebKit version that I am using on Qt 4.7 is numbered Qt WebKit 2.0.  It is included in Kubuntu 10.10.

[Ronald Clifford Buckman]

I confirmed this bug in both dwb and Uzbl running WebKitGTK+ 1.3.10.

[Martin Robinson]

(In reply to comment #7)
> I confirmed this bug in both dwb and Uzbl running WebKitGTK+ 1.3.10.

Try this uzbl config:

set font_size = 10
set minimum_font_size = 1

I believe that the issue in WebKitGTK+ is that the minimum font size is set to high by default.

[Ronald Clifford Buckman]

I set the minimum font size in the config file in uzbl to 1 and the font size to 10 and the blue links are still displayed on top of the menu bar like before.  The same display problem is now in Firefox 3.6 and SeaMonkey 2.0, although Arora running Qt WebKit 2.0 still displays the page as intended with the pull down menu working.

[Martin Robinson]

(In reply to comment #9)
> I set the minimum font size in the config file in uzbl to 1 and the font size to 10 and the blue links are still displayed on top of the menu bar like before.  The same display problem is now in Firefox 3.6 and SeaMonkey 2.0, although Arora running Qt WebKit 2.0 still displays the page as intended with the pull down menu working.

Are you building 1.3.10 from source? Can you try applying this patch and then running the GtkLauncher? After doing that and loading http://www.nba.com, the links show in the correct place.

diff --git a/Tools/GtkLauncher/main.c b/Tools/GtkLauncher/main.c
index ad666e2..d9f49aa 100644
--- a/Tools/GtkLauncher/main.c
+++ b/Tools/GtkLauncher/main.c
@@ -200,6 +200,10 @@ static GtkWidget* createWindow(WebKitWebView** outWebView)
     gtk_widget_set_name(window, "GtkLauncher");
 
     webView = WEBKIT_WEB_VIEW(webkit_web_view_new());
+    WebKitWebSettings* settings = webkit_web_view_get_settings(webView);
+    g_object_set(settings, "minimum-font-size", 1, NULL);
+    webkit_web_view_set_settings(webView, settings);
+
     uriEntry = gtk_entry_new();
 
     vbox = gtk_vbox_new(FALSE, 0);

[Ronald Clifford Buckman]

Created attachment 82740 [details]
nba.com on WebKitGTK+ 1.3.10 with patch applied.

Still a display problem.  Combination of poor coding on the home page and WebKitGTK (as well as Gecko on LInux and the KDE's Kwebkit component) not correcting it.

[Martin Robinson]

(In reply to comment #11)
> Created an attachment (id=82740) [details]
> nba.com on WebKitGTK+ 1.3.10 with patch applied.
> 
> Still a display problem.  Combination of poor coding on the home page and WebKitGTK (as well as Gecko on LInux and the KDE's Kwebkit component) not correcting it.

This is high confusing. Lowering the minimum font size fixes the issue for me. I see no other reason why this would break. What version of GTK+ are you using? What GTK+ theme?

[Martin Robinson]

(In reply to comment #11)
> Created an attachment (id=82740) [details]
> nba.com on WebKitGTK+ 1.3.10 with patch applied.
> 
> Still a display problem.  Combination of poor coding on the home page and WebKitGTK (as well as Gecko on LInux and the KDE's Kwebkit component) not correcting it.

Do you mind trying one more thing? Please drop the minimum font size down to 0 (with the patch I previously pasted for GtkLauncher) and see if that improves things.

[Sergio Villar Senin]

(In reply to comment #13)
> (In reply to comment #11)
> > Created an attachment (id=82740) [details] [details]
> > nba.com on WebKitGTK+ 1.3.10 with patch applied.
> > 
> > Still a display problem.  Combination of poor coding on the home page and WebKitGTK (as well as Gecko on LInux and the KDE's Kwebkit component) not correcting it.
> 
> Do you mind trying one more thing? Please drop the minimum font size down to 0 (with the patch I previously pasted for GtkLauncher) and see if that improves things.

Martin it works fine for me with your patch (font size to 0). If the minimum font size is 1 then the problem is still there (that's why it's always reproducible in epy as it isn't possible to drop the minimum value bellow 1).

[Martin Robinson]

Here's my proposal then:

1. Epiphany should allow minimum-font-size values of 1 from the preferences dialog and that should be the default. Maybe it should even hide the option now that you can adjust the default font size, which is a more appropriate knob.
2. We should make sure that minimum-logical-font-size can also be set to 0 and that they are both zero by default.

[Sergio Villar Senin]

(In reply to comment #15)
> Here's my proposal then:
> 
> 1. Epiphany should allow minimum-font-size values of 1 from the preferences dialog and that should be the default. Maybe it should even hide the option now that you can adjust the default font size, which is a more appropriate knob.

I agree. Chrome does not expose that setting, and in Firefox is an "advanced setting" default to 0.

> 2. We should make sure that minimum-logical-font-size can also be set to 0 and that they are both zero by default.

Againg agree

[Ronald Clifford Buckman]

I compiled and installed WebKitGTK+ 1.3.11, moved the minimum font size to 0 on dwb and the page displays as intended.  I confirm that any minimum font size above 0, causes the display and menu problem.

[Sergio Villar Senin]

I think this one was fixed, Martin?

[Martin Robinson]

Yep. Closing.