Last modified: 2014-05-16 17:55:10 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T47668, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45668 - Position of Search result is wrong hardcoded
Position of Search result is wrong hardcoded
Status: REOPENED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.21.x
All All
: Low minor (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-03-03 18:02 UTC by DaSch
Modified: 2014-05-16 17:55 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
example screenshot (29.15 KB, image/png)
2013-07-04 08:14 UTC, DaSch
Details

Description DaSch 2013-03-03 18:02:22 UTC
The position of search Suggestions somehow is a bit strange on other skins.
For example take a look at my maintained cavendish skin on http://www.wecowi.de
The box with sugesstions is far to right and trying to add margin does not work because mediawikis common css kills all tries.
Comment 1 Andre Klapper 2013-03-05 11:02:50 UTC
<div class="suggestions"> on your website has the style attributes
"top: 86.9px; bottom: auto; width: 276px; right: 139.2px"

On en.wikipedia.org, this is:
"top: 65.1px; bottom: auto; width: 198px; right: 6.91669px"

Looks like that's the place to locate and edit, wherever that comes from?
Comment 2 DaSch 2013-03-05 18:38:09 UTC
This is generated with javascript I guess. The problem is, that this Javascript works only correct if the buttons are below the textinput because the javascript looks for the form and makes the position after this and not from the position of the textinput.
Comment 3 DaSch 2013-07-04 08:14:38 UTC
Created attachment 12751 [details]
example screenshot

It's really sad, that nobody cares about such bad styled things, that prevent other developers in creating great new skins for mediawiki.
Comment 4 Bartosz Dziewoński 2013-07-10 12:42:32 UTC
You should be explicitly using a data-searchsuggest-expand-dir="left" (for LTR) or "right" (for RTL) attribute on your #searchInput. It's positioned in a way that's throwing off the automatic detection.

Or feel free to reopen this bug and submit a patch that will make it work in your skin and not break anything in all of the other skins it currently supports.


(In reply to comment #3)
> It's really sad, that nobody cares about such bad styled things, that prevent
> other developers in creating great new skins for mediawiki.

If by "great skins" you mean "your skin", then let me point out that the linked website is throwing JS errors as soonas someone visits it. I'm not going to be fixing it for you for free.
Comment 5 Bartosz Dziewoński 2013-07-10 12:50:06 UTC
Hmmmm actually, this might be caused by you using really weird margins on <body> and $.suggestions not expecting that.
Comment 6 Gerrit Notification Bot 2013-07-10 13:22:11 UTC
Change 72937 had a related patch set uploaded by Matmarex:
jquery.suggestions: Use document width instead of <body> width for position calculation

https://gerrit.wikimedia.org/r/72937
Comment 7 Gerrit Notification Bot 2013-08-25 13:43:58 UTC
Change 72937 merged by TheDJ:
jquery.suggestions: Use document width instead of <body> width for position calculation

https://gerrit.wikimedia.org/r/72937
Comment 8 Bartosz Dziewoński 2014-05-04 14:40:37 UTC
The fix seems to be causing noticeable issues, a revert was proposed: https://gerrit.wikimedia.org/r/#/c/130425/
Comment 9 Bartosz Dziewoński 2014-05-16 17:55:10 UTC
The revert was merged, reopening.

If anyone comes up with a solution that doesn't cause bug 54091 and bug 64233, I'd love to merge it.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links