Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#1484228 closed Bugs (fixed)

Searchbox doesn't work well in Konqueror

Reported by: Mikos Owned by:
Priority: 5 Milestone: 0.2-beta
Component: User Interface Version: git-master
Severity: minor Keywords:
Cc:

Description

I am using Konqueror (latest version from KDE 3.5.5) as my main browser, but there are some uncomfortable display bugs in RoundCube.

The first thing is search box, which isn't themed correctly (you can't see rounded corners).

The more important bug is when you click on mail folder (in left pane), folder is changed as it should be, but mouse pointer then looks like it is grabbing something (same way as if you grab some image from page).

Attachments (2)

grabbing.patch (1.1 KB) - added by runico 6 years ago.
patch for fix grabbing when press folders or buttons
quicksearch.patch (618 bytes) - added by runico 6 years ago.
fix for quicksearch bar

Download all attachments as: .zip

Change History (13)

comment:1 Changed 6 years ago by jshatch

I can confirm what is stated above (except for the spam...). I also add that the preview pane does not work in Konqueror. It shows half of the watermark image at first and after a few tries finally shows the message but only in the top half of the pane.

comment:2 Changed 6 years ago by thomasb

  • Priority changed from 1 to 5
  • Severity changed from major to minor

Minor. Could someone provide a patch?

comment:3 Changed 6 years ago by Rainbow

I use Konqueror too and the folder dragging was really annoying. It seems to be a Konqueror bug. This is a workaround:

diff -ur roundcubemail-nightly-20070318/program/steps/mail/func.inc roundcube-test/program/steps/mail/func.inc
--- roundcubemail-nightly-20070318/program/steps/mail/func.inc  2007-03-18 19:46:03.000000000 +0100
+++ roundcube-test/program/steps/mail/func.inc  2007-05-07 18:57:16.000000000 +0200
@@ -221,6 +221,7 @@

     $js_name = htmlspecialchars(JQ($folder['id']));
     $out .= sprintf('<li id="rcmbx%s" class="mailbox %s %s%s%s"><a href="%s&amp;_mbox=%s"'.
+                    ' onmousedown="return false"'.
                     ' onclick="return %s.command(\'list\',\'%s\')"'.
                     ' onmouseover="return %s.focus_mailbox(\'%s\')"' .
                     ' onmouseout="return %s.unfocus_mailbox(\'%s\')"' .

comment:4 Changed 6 years ago by Rainbow

The quicksearch box background is not displayed because of a CSS hack (What's it for? Maybe MSIE5 for Mac?). Removing it fixes Konqueror (the box is still not completely right, but a lot better), but I don't know what browser breaks now:

diff -ur roundcubemail-nightly-20070318/skins/default/mail.css roundcube-test/skins/default/mail.css
--- roundcubemail-nightly-20070318/skins/default/mail.css       2007-02-21 23:53:37.000000000 +0100
+++ roundcube-test/skins/default/mail.css       2007-05-07 20:33:53.000000000 +0200
@@ -559,14 +559,6 @@
   border: none;
 }

-
-/*\*/
-html>body*#quicksearchbar[id$="quicksearchbar"]:not([class="none"]) { background-image: none; }
-html>body*#quicksearchbar[id$="quicksearchbar"]:not([class="none"]) a { top: 5px; }
-html>body*#quicksearchbar[id$="quicksearchbar"]:not([class="none"]) #quicksearchbox { width: 180px; top:0px; right: 1px; left: auto; }
-/**/
-
-
 #rcversion
 {
   position: absolute;

Changed 6 years ago by runico

patch for fix grabbing when press folders or buttons

comment:5 Changed 6 years ago by runico

I have attached a patch against svn (r774/svn)

Changed 6 years ago by runico

fix for quicksearch bar

comment:6 Changed 6 years ago by runico

Patch for fixing quicksearch bar. I have tried with konqueror, firefox and internet explorer and it's shown well with the patch.

comment:7 Changed 5 years ago by seansan

  • Milestone set to 0.1.1
  • Summary changed from RoundCube doesn't work well in Konqueror to Searchbox doesn't work well in Konqueror

Review in 0.1.1

comment:8 Changed 5 years ago by robin

Proposed quicksearch.patch breaks in Safari.

comment:9 Changed 5 years ago by alec

grabbing problem fixed in [4da1d7c6].

comment:10 Changed 5 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

... the rest is a duplicate of #1484841

comment:11 Changed 5 years ago by alec

  • Component changed from Client Scripts to User Interface
Note: See TracTickets for help on using tickets.