Page MenuHomePhabricator

VisualEditor: Message <visualeditor-feedback-link> should be used in content language
Closed, ResolvedPublic

Description

Having users using different languages to post feedback on different pages on the same site makes it difficult to manage.


Version: unspecified
Severity: minor

Details

Reference
bz47730

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:43 AM
bzimport set Reference to bz47730.

(Is this a bug in the core MW feedback tool?)

I'm not sure what "core MW feedback tool" means, but I was talking about the feedback feature for [[Wikipedia:VisualEditor/Feedback]].

(In reply to comment #2)

I'm not sure what "core MW feedback tool" means, but I was talking about the
feedback feature for [[Wikipedia:VisualEditor/Feedback]].

Yes, this is a core MediaWiki feature, it just happens to be used by VisualEditor:

resources/mediawiki/mediawiki.feedback.js

But there is:

modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js:

		'title': new mw.Title( ve.msg( 'visualeditor-feedback-link' ) ),

shouldn't something like:

new mw.Title( ve.msg( 'visualeditor-feedback-link' ).inContentLanguage() )

be used?

Yes, that would fix this particular problem; my question is more as to whether the core tool should default for target link to be in the content language.

(In reply to comment #5)

Yes, that would fix this particular problem; my question is more as to
whether
the core tool should default for target link to be in the content language.

Based on what I see in the usage given above, you're giving a Title to core feedback tool, which points to a certain page, instead of a message name to core with feedback page name in that message.

Related URL: https://gerrit.wikimedia.org/r/64498 (Gerrit Change I3804e6f91015ef9b8ae5196efb37475d714cd7d5)

  • Bug 48645 has been marked as a duplicate of this bug. ***

pl.wiki is on 1.22wmf8 but my feedback was posted to a non-existing page with Italian title (I have Italian UI), [[pl:Wikipedia:VisualEditor/Commenti]]: https://pl.wikipedia.org/w/index.php?title=Wikipedia:VisualEditor/Commenti&oldid=36959657

(In reply to comment #10)

pl.wiki is on 1.22wmf8 but my feedback was posted to a non-existing page with
Italian title (I have Italian UI), [[pl:Wikipedia:VisualEditor/Commenti]]:
https://pl.wikipedia.org/w/index.php?title=Wikipedia:VisualEditor/
Commenti&oldid=36959657

Confirmed, this is also happening to me. Did the code get broken somehow?

(In reply to comment #11)

(In reply to comment #10)

pl.wiki is on 1.22wmf8 but my feedback was posted to a non-existing page with
Italian title (I have Italian UI), [[pl:Wikipedia:VisualEditor/Commenti]]:
https://pl.wikipedia.org/w/index.php?title=Wikipedia:VisualEditor/
Commenti&oldid=36959657

Confirmed, this is also happening to me. Did the code get broken somehow?

Somewhere between https://gerrit.wikimedia.org/r/#/c/68216/ and its parent (I can't seem to find the commit that added it) the message was added in VisualEditor.php's resource module definition. Which caused the message from VisualEditorMessagesModule (in content-language) to be overridden at run time by the one from ResourceLoader for user language.

Change 74064 had a related patch set uploaded by Krinkle:
Remove 'visualeditor-feedback-link' from RL module

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

Fixed. Sorry about this. Will be deployed this afternoon.

Change 74064 merged by jenkins-bot:
Remove 'visualeditor-feedback-link' from RL module

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