Closed
Bug 482347
Opened 15 years ago
Closed 15 years ago
make SHOWFOR UA detection work with both 3.1 and 3.5
Categories
(support.mozilla.org :: Knowledge Base Software, task)
support.mozilla.org
Knowledge Base Software
Tracking
(Not tracked)
VERIFIED
FIXED
1.0.1
People
(Reporter: cilias, Assigned: ecooper)
References
()
Details
(Whiteboard: sumo_only showfor)
Attachments
(1 file)
570 bytes,
patch
|
laura
:
review+
|
Details | Diff | Splinter Review |
Firefox 3.1 is being reversioned to Firefox 3.5. SHOWFOR should be updated for 3.5, but we don't want to break UA detection on builds labeled as 3.1. So if the UA is either 3.1 or 3.5, Firefox 3.5 content will be displayed.
Updated•15 years ago
|
Target Milestone: --- → 1.0.1
Updated•15 years ago
|
Assignee: nobody → smirkingsisyphus
Assignee | ||
Comment 1•15 years ago
|
||
Firefox 3.1 and 3.5 will be pointed to 3.5. Testing this will need the patch from bug 482342. Or you can test by running 'showfor_detect_browser()' in the Firebug console.
Attachment #370332 -
Flags: review?(laura)
Updated•15 years ago
|
Attachment #370332 -
Flags: review?(laura) → review+
Assignee | ||
Comment 2•15 years ago
|
||
r24107/r24108
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 3•15 years ago
|
||
Chris, do you have any good articles that use this, so I can test? Would you mind taking a look too? Thanks!
Comment 4•15 years ago
|
||
added testing link to URL field
Comment 5•15 years ago
|
||
Krupa; mind giving this a shot?
Comment 6•15 years ago
|
||
So far I've tested: * On Windows: Firefox 2.0.0.20, Firefox 3.0, Firefox 3.0.5, Firefox 3.0.8, Firefox 3.1 beta 3, Shiretoko * On Mac: Firefox 3.0.8, Shiretoko All good so far; I note that non-Firefox-based browsers default to 3.0; is that expected?
Comment 7•15 years ago
|
||
Linux: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3) Gecko/20090305 Firefox/3.1b3 is good too.
Comment 8•15 years ago
|
||
(In reply to comment #6) > All good so far; I note that non-Firefox-based browsers default to 3.0; is that > expected? Yes, that's expected. Once 3.5 is officially released, that should change though.
Comment 9•15 years ago
|
||
Verified FIXED: I've checked 3.1 beta 3 on Mac as well as 2.0.0.20, too.
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
Whiteboard: sumo_only showfor
You need to log in
before you can comment on or make changes to this bug.
Description
•