Closed Bug 631152 Opened 10 years ago Closed 10 years ago

Search autocomplete popup opens in the wrong position in RTL mode

Categories

(Firefox :: Search, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 5

People

(Reporter: ehsan, Assigned: ehsan)

References

Details

(Keywords: rtl)

Attachments

(5 files, 2 obsolete files)

Attached image Screenshot
See the screenshot.
blocking2.0: --- → ?
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.
Whiteboard: [DUPEME]
I could reproduce this on Hebrew trunk builds on all platforms.
OS: Mac OS X → All
Hardware: x86 → All
Attached patch Patch (v1) (obsolete) — Splinter Review
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.
Whiteboard: [DUPEME]
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-
Attached image With patch, LTR
Attached image With patch, RTL
Attached patch Patch (v2) (obsolete) — Splinter Review
Attachment #509525 - Attachment is obsolete: true
Attachment #509634 - Flags: review?(gavin.sharp)
Attachment #509634 - Flags: approval2.0?
Actually it's not a regression...
blocking2.0: ? → ---
Duplicate of this bug: 405968
(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.
Attachment #509634 - Flags: approval2.0?
Gavin: ping?
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+
Attached patch For check-inSplinter Review
Attachment #509634 - Attachment is obsolete: true
http://hg.mozilla.org/mozilla-central/rev/be9814a5a456
Status: ASSIGNED → RESOLVED
Closed: 10 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
Target Milestone: Firefox5 → Firefox 5
Duplicate of this bug: 666749
You need to log in before you can comment on or make changes to this bug.