Closed
Bug 78545
Opened 24 years ago
Closed 24 years ago
Tinderbox 'Show checkins' URLs load in the iframe, not the main window
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
VERIFIED
FIXED
People
(Reporter: sfraser_bugs, Assigned: mscott)
References
Details
(Keywords: regression)
For a while now, clicking on someone's name in the Guilty column on Tinderbox,
then choosing 'Check-ins in the last 24 hours' in the resulting floating iframe
loads the resulting page into that iframe, rather than the main browser window.
This seems like a url targetting issue.
Updated•24 years ago
|
Severity: normal → major
Comment 1•24 years ago
|
||
not sure the status on moz0.9, but how soon could this get fixed? pretty bad
that Show Checkins isn't working anymore.
Comment 2•24 years ago
|
||
chatted w/mcafee --this likely isn't a tbox source issue, but an iframe issue.
he's seen it for some time on the commercial tbox. but, oddly, i cannot repro
this using 6.01 [linux, both modern and classic are fine, so not themes here
either].
Comment 3•24 years ago
|
||
mcafee sees this in gtkEmbed, so punting over to layout.
Assignee: pchen → karnaze
Component: XP Apps → Layout
QA Contact: sairuh → petersen
Reporter | ||
Comment 4•24 years ago
|
||
Hrmm, I think this is mscott or someone like danm. We're loading the URL in the
wrong docShell, which is outside of layout.
Comment 6•24 years ago
|
||
This worksforme with 050209 0_9_BRANCH build on Mac OS 9. It also seems fine on
win32 and linuc 0_9_BRANCH builds. I tested by loading tinderbox and clicking
on a name in the guilty column. An iframe popup came up and I clicked each of
the checkin links. All of them resulted in the bonsai CVS Checkins page loading
in the browser window and not the iframe. Perhaps this regressed after the
branch was cut.
Updated•24 years ago
|
OS: Mac System 8.5 → All
Reporter | ||
Comment 7•24 years ago
|
||
I think this should go to rpotts.
Assignee | ||
Comment 8•24 years ago
|
||
I believe this was fixed yesterday. Using today's latest release builds this is
working for me. I just tested on windows. I'll try Mac next.
Comment 9•24 years ago
|
||
yes this looks fixed on linux, too.
Assignee | ||
Comment 10•24 years ago
|
||
marking fixed.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•