Closed
Bug 651225
Opened 10 years ago
Closed 10 years ago
Change showfor semantics and add = operator
Categories
(support.mozilla.org :: Knowledge Base Software, task)
support.mozilla.org
Knowledge Base Software
Tracking
(Not tracked)
VERIFIED
FIXED
2.9
People
(Reporter: jsocol, Assigned: erik)
Details
(Whiteboard: [showfor])
For Firefox versions, we need to change the behavior of showfor in the following ways: * {for fx4} now means "For Firefox 4 *and up*" * {for =fx4} now means "For Firefox 4 *only*" * {for fx35} is a special case that still means Firefox 3.5-3.6. Future versions will follow the fx4 pattern, e.g.: {for fx5} will mean "For Firefox 5 and up" This will involve a whole lot of python and JS hacking. Team, assemble!
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → erik
Reporter | ||
Comment 1•10 years ago
|
||
We'll also need to special case {for fx3} to be =fx3, for now. See bug 651220 comment 6.
Assignee | ||
Comment 2•10 years ago
|
||
See http://etherpad.mozilla.com:9000/2011-showfor.
Comment 3•10 years ago
|
||
Looking at the fx development schedule, we'll have a few articles that we'll need to update for fx5 by the end of May. It will be great to have this by then or I guess the plan B would be to call out Fx5 differences in the Fx4 articles.
Comment 4•10 years ago
|
||
Ideally this would be great to have in 2.8.4 if possible. That would give us a week to finalize articles with this markup before giving them to localizers.
Updated•10 years ago
|
Target Milestone: 2011Q2 → 2.8.4
Reporter | ||
Comment 5•10 years ago
|
||
2.8.4 is not remotely realistic. It's freezing on Wednesday morning and this is a huge, huge change that hasn't--or has barely--gotten started.
Target Milestone: 2.8.4 → 2011Q2
Assignee | ||
Updated•10 years ago
|
Target Milestone: 2011Q2 → 2.9
Assignee | ||
Comment 6•10 years ago
|
||
Done in http://github.com/jsocol/kitsune/commit/472cec7.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 7•10 years ago
|
||
Verified correct behavior for showfor fx 35, 4, 5- unable to verify fx3 as the menu item has been removed [will re-verify if another method for testing comes up].
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•