Last modified: 2014-02-22 02:04:49 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 T63321, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 61321 - Beta tag ({{int:visualeditor-beta-appendix}}) should not always be displayed in the guider description
Beta tag ({{int:visualeditor-beta-appendix}}) should not always be displayed ...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
GuidedTour (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Matthew Flaschen
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-13 17:27 UTC by Steven Walling
Modified: 2014-02-22 02:04 UTC (History)
7 users (show)

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


Attachments
Francais (82.34 KB, image/png)
2014-02-13 17:27 UTC, Steven Walling
Details

Description Steven Walling 2014-02-13 17:27:07 UTC
Created attachment 14579 [details]
Francais

Edit beta is specific to English Wikipedia but we hard coded it in the tour text for GettingStarted, so it is showing up incorrectly in translated versions such as French etc.
Comment 1 Matthew Flaschen 2014-02-14 01:07:08 UTC
It's easy to detect if this is enabled (mw.config.get( 'wgVisualEditorConfig' ).tabMessages).  However, I need to decide whether to make separate messages or substitute it in.

Is this an acceptable use of a parameter (passing either '{{int:vector-view-edit}} {{int:visualeditor-beta-appendix}}"' or '{{int:vector-view-edit}}' depending on configuration?

Or should I make separate messages?
Comment 2 Nemo 2014-02-14 07:29:01 UTC
(In reply to Matthew Flaschen from comment #1)
> It's easy to detect if this is enabled (mw.config.get(
> 'wgVisualEditorConfig' ).tabMessages).  However, I need to decide whether to
> make separate messages or substitute it in.

Can you pass two messages to it directly?

> 
> Is this an acceptable use of a parameter (passing either
> '{{int:vector-view-edit}} {{int:visualeditor-beta-appendix}}"' or
> '{{int:vector-view-edit}}' depending on configuration?
> 
> Or should I make separate messages?

If the message(s) are just '{{int:vector-view-edit}} {{int:visualeditor-beta-appendix}}' it's not less lego to have two. You can pass to the tab a message '{{int:vector-view-edit}}$1" where $1 is nothing or " {{int:visualeditor-beta-appendix}}".
Comment 3 Matthew Flaschen 2014-02-14 18:31:07 UTC
(In reply to Nemo from comment #2)
> (In reply to Matthew Flaschen from comment #1)
> > It's easy to detect if this is enabled (mw.config.get(
> > 'wgVisualEditorConfig' ).tabMessages).  However, I need to decide whether to
> > make separate messages or substitute it in.
> 
> Can you pass two messages to it directly?

Sorry, I'm a little unclear what you mean.  What would the two messages be, and what would I pass them to?

> If the message(s) are just '{{int:vector-view-edit}}
> {{int:visualeditor-beta-appendix}}' it's not less lego to have two. You can
> pass to the tab a message '{{int:vector-view-edit}}$1" where $1 is nothing
> or " {{int:visualeditor-beta-appendix}}".

I'm not controlling the text of the tab (sorry, the bug title was misleading).  I'm controlling the text of the guider (a kind of modal popup), which refers to the text of the tab.

However, I think your suggestion of using a parameter just for the second part could still apply.
Comment 4 Matthew Flaschen 2014-02-14 18:32:56 UTC
See the attachment for the issue.  Note how the guider says "Modifier bêta", but the actual tab just says "Modifier".  We want to fix the guider to check the configuration and always match the tab.
Comment 5 Matthew Flaschen 2014-02-20 23:07:32 UTC
This is in a firsteditve message in the GuidedTour extension.
Comment 6 Gerrit Notification Bot 2014-02-21 06:01:15 UTC
Change 114698 had a related patch set uploaded by Mattflaschen:
Use VisualEditor configuration for appendices (e.g. beta).

https://gerrit.wikimedia.org/r/114698
Comment 7 Gerrit Notification Bot 2014-02-21 16:20:33 UTC
Change 114698 merged by Phuedx:
Use VisualEditor configuration for appendices (e.g. beta).

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

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


Navigation
Links