Closed Bug 631152 Opened 11 years ago Closed 11 years ago
Search autocomplete popup opens in the wrong position in RTL mode
48.47 KB, image/png
18.14 KB, image/png
24.68 KB, image/png
25.17 KB, image/png
3.14 KB, patch
|Details | Diff | Splinter Review|
See the screenshot.
Ehsan, does it happen on all platforms? Would be good to know between which beta releases it has been regressed.
Version: unspecified → Trunk
I've tested on both linux and win32 platoforms. The popup appears in its right place there. See the second screenshot.
So this is an OS X only issue? Ehsan, are you using multiple screens?
(In reply to comment #4) > So this is an OS X only issue? Ehsan, are you using multiple screens? I haven't tested on other platforms, and no, I wasn't been using multiple screen.
Also happens in 3.6.13 builds of ar on OS X. I'm sure this is a dupe of another bug.
I could reproduce this on Hebrew trunk builds on all platforms.
OS: Mac OS X → All
Hardware: x86 → All
We shouldn't really think that we can position the popup better than the xul popup manager code!
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #509525 - Flags: review?(gavin.sharp)
Henrik, please dupe any other bugs on this you can find against this one, as it has a patch.
Comment on attachment 509525 [details] [diff] [review] Patch (v1) This method really needs to be synced with autocomplete.xml's _openAutocompletePopup (and I don't think you want to be removing the width setting). We may be able to do that by just removing this override entirely, but we need to make sure not to regress bug 398020 comment 24 (aligning the popup with the text field itself, rather than the engine selector dropdown).
Attachment #509525 - Flags: review?(gavin.sharp) → review-
(In reply to comment #14) > Actually it's not a regression... Are you sure about that? In the second attachment, taken from a linux64 build of fx4 beta that was built a couple of weeks ago, the popup appears in its proper place. However, I have just made a new linux64 build where I've found that the popup is shifted from its expected place. It seems that the cause was probably landed in mozilla-central in that short period.
Do you have a closer time range for Linux?
(In reply to comment #16) > (In reply to comment #14) > > Actually it's not a regression... > > Are you sure about that? In the second attachment, taken from a linux64 build > of fx4 beta that was built a couple of weeks ago, the popup appears in its > proper place. However, I have just made a new linux64 build where I've found > that the popup is shifted from its expected place. It seems that the cause was > probably landed in mozilla-central in that short period. Yes, I'm sure (judging by reading the faulty code, and how long it goes back, and actually testing on 3.6!). Please note that according to the exact place of the Firefox window on the screen, there is a slight possibility that you would see the autocomplete popup in the right position, but that's an exception, and moving the window on the screen reveals the problem.
Comment on attachment 509634 [details] [diff] [review] Patch (v2) Gavin: I think this is a nice fix to take for RTL locales. It's not a blocker by any means, but if you had a few spare cycles, I'd really appreciate if you can take a look at this.
Comment on attachment 509634 [details] [diff] [review] Patch (v2) Can you add a comment to the top of this method along the lines of: "This method overrides the autocomplete binding's openPopup (essentially duplicating the logic from the autocomplete popup binding's openAutocompletePopup method), modifying it so that the popup is aligned with the inner textbox, but sized to not extend beyond the search bar border."
Attachment #509634 - Flags: review?(gavin.sharp) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox4.2
Verified on Mozilla/5.0 (Windows NT 6.1; rv:2.2a1pre) Gecko/20110403 Firefox/4.2a1pre Search autocomplete popup opens in the wright position.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.