Last modified: 2014-07-16 13:58: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 T69244, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67244 - Adding site links is broken
Adding site links is broken
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
unspecified
All All
: Highest critical (vote)
: ---
Assigned To: Wikidata bugs
u=dev c=frontend p=5 s=2014-07-01
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-28 11:25 UTC by Aude
Modified: 2014-07-16 13:58 UTC (History)
4 users (show)

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


Attachments

Description Aude 2014-06-28 11:25:46 UTC
(with master Wikibase) there is no "add" button for site links when javascript is enabled.

http://snag.gy/XcZ09.jpg

When javascript is disabled, the css for the <td> right border in the add/edit links column is a bit broken.

http://snag.gy/nEFlz.jpg

the site links broke with https://gerrit.wikimedia.org/r/#/c/140088/ which added a span tag and changed which element has the .wb-editsection.

https://gist.github.com/filbertkm/9014f962ebcffcd03cf8/revisions

The css and javascript is not adjusted accordingly.

Furthermore, this is a breaking change that needs a DEPLOY note to purge the parser cache.  Purging the parser cache is something we really don't want to do very often and only if absolutely needed.

If possible and not too difficult, I prefer the css / javascript to be adjusted with backwards compatibility to work with either version of the html and not have to purge the parser cache for this.
Comment 1 Gerrit Notification Bot 2014-06-28 11:29:15 UTC
Change 142523 had a related patch set uploaded by Aude:
Partial revert of I2e7bd49b to fix add site links bug

https://gerrit.wikimedia.org/r/142523
Comment 2 Gerrit Notification Bot 2014-06-30 13:17:41 UTC
Change 143022 had a related patch set uploaded by Aude:
Fix site links 'add' links button in javascript

https://gerrit.wikimedia.org/r/143022
Comment 3 Aude 2014-07-01 17:53:04 UTC
now broken on wikidata beta:

http://wikidata.beta.wmflabs.org/wiki/Q27843
Comment 4 Gerrit Notification Bot 2014-07-02 10:21:26 UTC
Change 143584 had a related patch set uploaded by Adrian Lang:
Fix site links edit sections

https://gerrit.wikimedia.org/r/143584
Comment 5 Gerrit Notification Bot 2014-07-02 14:23:57 UTC
Change 143022 abandoned by Aude:
Fix site links 'add' links button in javascript

https://gerrit.wikimedia.org/r/143022
Comment 6 Gerrit Notification Bot 2014-07-03 11:18:12 UTC
Change 142523 abandoned by Aude:
Partial revert of I2e7bd49b to fix add site links bug

Reason:
no longer needed

https://gerrit.wikimedia.org/r/142523
Comment 7 Gerrit Notification Bot 2014-07-03 11:23:55 UTC
Change 143584 merged by jenkins-bot:
Fix site links edit sections

https://gerrit.wikimedia.org/r/143584

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


Navigation
Links