Hoo man has uploaded a new change for review.

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

Change subject: Fix selector for linkitem init
......................................................................

Fix selector for linkitem init

The span either has wb-langlinks-edit if the page already is
connected to an item or wb-langlinks-add if it's not.

But the span never has wbc-editpage which is now being set on
the a within the span. If this anchor is being used, we will
have two nested links (only one visible though) with two pencils
next to them.

Bug: 65001
Change-Id: Iac189e7513fdb366aa418dac891f61b51a898354
---
M client/resources/wikibase.client.linkitem.init.js
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/16/132116/1

diff --git a/client/resources/wikibase.client.linkitem.init.js 
b/client/resources/wikibase.client.linkitem.init.js
index c8948a8..fe443a1 100644
--- a/client/resources/wikibase.client.linkitem.init.js
+++ b/client/resources/wikibase.client.linkitem.init.js
@@ -59,8 +59,7 @@
                        return;
                }
 
-               // @note .wbc-editpage is deprecated
-               $( '.wbc-editpage, .wb-langlinks-add' ).eq( 0 )
+               $( '.wb-langlinks-edit, .wb-langlinks-add' ).eq( 0 )
                .empty()
                .append(
                        $( '<a>' )

-- 
To view, visit https://gerrit.wikimedia.org/r/132116
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iac189e7513fdb366aa418dac891f61b51a898354
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Hoo man <h...@online.de>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to