Bug 172932 - konqueror is broken with jcarousel
Summary: konqueror is broken with jcarousel
Status: RESOLVED WORKSFORME
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (other bugs)
Version First Reported In: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Bugs
URL:
Keywords:
: 174250 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-10-16 10:40 UTC by Anders Lund
Modified: 2021-03-21 08:32 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Lund 2008-10-16 10:40:30 UTC
Version:           4.1.2 (KDE 4.1.2) (using 4.1.2 (KDE 4.1.2), Kubuntu packages)
Compiler:          gcc
OS:                Linux (x86_64) release 2.6.24-16-generic

In kde 3, konqueror works with jcarousel (http://sorgalla.com/jcarousel) - in kde 4 it does not, trying to load the examples at sorgella.com it simply freezes at 25 %, requiring a kill to stop the application.
Comment 1 Anders Lund 2008-10-17 09:06:33 UTC
Setting the UA to 'Safari 2.0 on Mac OS X' makes it work, But I repeat that it works with kde 3 konqueror, so that should NOT be nessecary.
Comment 2 Frank Reininghaus 2008-10-18 11:19:02 UTC
I can confirm the problem in 4.1.2 (Kubuntu), for example with the "Simple carousel". However, trunk rev. 871890 (with default UA) shows a dialog box "jCarousel: No width/height set for items. This will cause an infinite loop. Aborting...". Konqueror doesn't freeze then, but the carousel doesn't work either.

Maybe it's a bug in 4.1.x that is fixed already in trunk?
Comment 3 Anders Lund 2008-10-18 11:40:43 UTC
The reason it works using the safari ua string is that the script looks for safari and calculates the width. Maybe it was worth trying with other webkit based browsers?

I'm not familliar with the internals of HTML rendering and javascript engines, but I assume that gecko and MSHTML calculates such values on demand.
Comment 4 Frank Reininghaus 2008-10-18 14:18:47 UTC
I tried different UA settings in trunk and experimented a bit. On the first load of the "Simple carousel" example (middle-clicked the link to open in another tab), I get an error message for Konqueror, Firefox and IE, but no error for Safari. The carousel doesn't work. Loading the example a second time in a new tab, there are no error messages, and the carousel works fine for any UA setting. Strange...

4.1.2 (Kubuntu) freezes for me for Konqueror, Firefox and IE UA settings, but not for Safari.

I don't know where the problem is, but I'll change the component to 'KHTML' because that's probably closer to it than 'general'.
Comment 5 Anders Lund 2008-10-18 15:16:25 UTC
As I said, the script contains some special cases for safari. Which is why I wonder how other webkit browsers behave (and what their UA strings are)
Comment 6 Maksim Orlovich 2008-10-18 20:51:55 UTC
We're getting renderer:0 when offsetWidth is called... And oddly 
enough an alert "fixes" it, which ought not matter if it's just display:none..
Comment 7 Frank Reininghaus 2008-11-07 21:50:00 UTC
*** Bug 174250 has been marked as a duplicate of this bug. ***
Comment 8 Justin Zobel 2021-03-21 00:25:56 UTC
Thank you for the bug report.

As this report hasn't seen any changes in 10 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.
Comment 9 Anders Lund 2021-03-21 08:32:34 UTC
I don't think this has any relevance any longer.