Closed
Bug 714577
Opened 13 years ago
Closed 13 years ago
layout is broken - http://www.jms-go.com/ELP-2011-autumn/index4.html
Categories
(Tech Evangelism Graveyard :: Japanese, defect)
Tech Evangelism Graveyard
Japanese
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alice0775, Unassigned)
References
()
Details
(Keywords: regression)
Reproducible: Always
Steps to Reproduce:
1. Open URL
Actual Results:
layout is broken
Expected Results:
layout should not be broken
Regression window(m-c)
Works:
http://hg.mozilla.org/mozilla-central/rev/1853104e4525
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0a1) Gecko/20110417 Firefox/6.0a1 ID:20110417030531
Fails:
http://hg.mozilla.org/mozilla-central/rev/c551b62cf2e8
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0a1) Gecko/20110416 Firefox/6.0a1 ID:20110417022149
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=1853104e4525&tochange=c551b62cf2e8
Triggered by;
c551b62cf2e8 Ms2ger — Bug 585877 - Remove support for document.width and document.height; r=jonas sr=jst * * * try: -b do -p all -u mochitest-o -t none
It seem to be TE.
http://www.jms-go.com/js/jms.js
function GetHeight(Y,NAME)
{
var app = navigator.appName.charAt(0);
if(navigator.userAgent.indexOf('Safari') != -1){
document.getElementById(Y).height = parent.frames[NAME].document.body.scrollHeight + 10;
}else if (app == "N")
{
document.getElementById(Y).height = parent.frames[NAME].document.height +10;
}
else
{
document.getElementById(Y).height = parent.frames[NAME].document.body.scrollHeight + 10;
}
}
I think they should just remove the two if clauses
if(...'Safari'){} and if(app == N){}
Alice, can you contact them?
Assignee: nobody → japanese
Component: DOM → Japanese
OS: Windows XP → All
Product: Core → Tech Evangelism
QA Contact: general → japanese
Hardware: x86 → All
Version: Trunk → unspecified
Reporter | ||
Comment 2•13 years ago
|
||
(In reply to j.j. from comment #1)
> Alice, can you contact them?
Done,
I e-mailed director@tacti.co.jp
Reporter | ||
Comment 3•13 years ago
|
||
The site was fixed today.
And I confirmed this was fixed on Firefox9.0.1 to Nightly12.0a1.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•