Closed
Bug 127969
Opened 23 years ago
Closed 23 years ago
[DEP][FIX]Print-preview frames orientation-switching issues
Categories
(Core :: Print Preview, defect, P3)
Tracking
()
VERIFIED
DUPLICATE
of bug 122048
mozilla1.0
People
(Reporter: megabyte, Assigned: rods)
References
()
Details
(Whiteboard: [adt3])
Attachments
(1 file)
(deleted),
patch
|
dcone
:
review+
attinasi
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
When switching between portrait and landscape orientation, a frames page
disappears, but the frame dividers stay in place. Also, I've seen it render the
NOFRAMES content on the second page in plaintext HTML.
2002022503/WinXP
Assignee | ||
Comment 1•23 years ago
|
||
Needs more investigation.
Assignee | ||
Comment 2•23 years ago
|
||
1) Added a new method to nsIWebBrowserPrint iface that enables you to get the
current PS being used to print or PP
2) Change the PP toolbar code to use the current PS instead of the global
Assignee | ||
Updated•23 years ago
|
Keywords: review
Summary: Print-preview frames orientation-switching issues → [FIX]Print-preview frames orientation-switching issues
Updated•23 years ago
|
Comment 3•23 years ago
|
||
Comment on attachment 72618 [details] [diff] [review]
patch
r=dcone
Attachment #72618 -
Flags: review+
Comment 5•23 years ago
|
||
Comment on attachment 72618 [details] [diff] [review]
patch
sr=attinasi
Attachment #72618 -
Flags: superreview+
Comment 6•23 years ago
|
||
Comment on attachment 72618 [details] [diff] [review]
patch
a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #72618 -
Flags: approval+
Assignee | ||
Comment 7•23 years ago
|
||
fixed
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 9•23 years ago
|
||
Nope, sorry, I still see this with build 2002031008/WinXP on all frames pages.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 10•23 years ago
|
||
See what? I see that framesets are not working in Landscape but I do not see the
switching back to portrait once it is in landscape and you hit scale.
Assignee | ||
Comment 11•23 years ago
|
||
The reason the sub-docs don't show up in landscape is that for some reason PP is
being called a second time (when in landscape) but not when in Portrait, this
second call is a new bug.
When in Portrait, if you hit the scale button or change a PrintSetting to cause
an additional change to the PP the sub-docs dissappear, this is caused by Bug
108509 and will soon be fixed by moving the loading of sub-docs to content (a
jst bug).
So, if when viewing a doc in PP in landscape mode and you hit the scale button,
if it stays in lanscape then please close this bug. This works for me.
Reporter | ||
Comment 12•23 years ago
|
||
All I do is hit the Landscape button (being in portrait mode) and everything
breaks. Just as I originally reported.
Assignee | ||
Comment 13•23 years ago
|
||
sujay, Are you seeing this on a page without framesets?
Comment 14•23 years ago
|
||
Rod, for frames, there is definitely a problem as the reporter
originaly states. I get the same result the reporter gets using
his test page.
For a non-frame site(abcnews.com), I can switch back and
forth between landscape and portrait mode w/o problems.
I'm using 3/12 trunk commercial build on win 98
Assignee | ||
Comment 15•23 years ago
|
||
Ok, so I fixed a different problem with my previous check in for this bug.
This is Frameset issue as I stated in comment #11 AND the fact that it gets
asked to PP twice. I will make this dependent on Bug 109509, which when jst
checks in his content/sub-docs fix, this should be fixed.
Assignee | ||
Updated•23 years ago
|
Summary: Print-preview frames orientation-switching issues → [DEP]Print-preview frames orientation-switching issues
Assignee | ||
Updated•23 years ago
|
Summary: [DEP]Print-preview frames orientation-switching issues → [DEP][FIX]Print-preview frames orientation-switching issues
Comment 16•23 years ago
|
||
This depends on bug 52334 not 108509
Assignee | ||
Comment 18•23 years ago
|
||
I am dup'ing this to Bug 122048 because the patch in that bug fixes this bug
also. This bug will also be fixed when Bug 52334 gets checked in. So we are well
covered and I want to take this off the radar.
*** This bug has been marked as a duplicate of 122048 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → DUPLICATE
Comment 19•23 years ago
|
||
verified...this is fixed now in today's 3/27 build.
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 20•23 years ago
|
||
Filed bug 133808 and bug 133809 to hit on some related issues.
You need to log in
before you can comment on or make changes to this bug.
Description
•