Applets sized using % do not resize with browser window

VERIFIED FIXED

Status

Core Graveyard
Java: OJI
VERIFIED FIXED
16 years ago
7 years ago

People

(Reporter: Ed Buck, Assigned: Patrick C. Beard)

Tracking

Trunk
PowerPC
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 2 obsolete attachments)

(Reporter)

Description

16 years ago
As of MRJPlugin version 1.0d7, it was possible to specify applet size using 
percentages (i.e height=100% width=100%).  However, when percentages are used, 
the expected behavior is for the applet to automatically resize/repaint when 
the browser window is resize.  With the latest version of the MRJPlugin 
(1.0b2), applets do not resize with the window.  The user must refresh the 
browser window to get the applet to resize.
(Assignee)

Comment 1

16 years ago
Brian, looks like we need to add code to call SizeJavaControl whenever the
width/height of the plugin window changes.
Assignee: beard → bnesse

Comment 2

16 years ago
Ed, do you have a test case for this?
Blocks: 88870
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
(Reporter)

Comment 3

16 years ago
Created attachment 76025 [details]
Java source file for test case
(Reporter)

Comment 4

16 years ago
Created attachment 76026 [details]
HTML file for test case
(Reporter)

Comment 5

16 years ago
Created attachment 76027 [details]
Class file for test case

Comment 6

16 years ago
Created attachment 76064 [details] [diff] [review]
Proposed patch

Patch to call SizeJavaControl() before calling MoveAndClipJavaControl() (which
seems a bit silly IMO... but it does work.

Comment 7

16 years ago
Comment on attachment 76064 [details] [diff] [review]
Proposed patch

if (silly && works)
  r=sdagley
Attachment #76064 - Flags: review+
(Assignee)

Comment 8

16 years ago
Comment on attachment 76064 [details] [diff] [review]
Proposed patch

you should use the width & height fields from the nsPluginWindow struct.
Attachment #76064 - Flags: needs-work+

Comment 9

16 years ago
Created attachment 76121 [details] [diff] [review]
Revised patch

Patch addressing beards comments. Patch also removes surrounding ifdefs and
dead code.
Attachment #76064 - Attachment is obsolete: true

Comment 10

16 years ago
Created attachment 76131 [details] [diff] [review]
Patch without extra crap from other patches.

Updated

16 years ago
Attachment #76121 - Attachment is obsolete: true

Comment 11

16 years ago
Comment on attachment 76131 [details] [diff] [review]
Patch without extra crap from other patches.

r=sdagley
Attachment #76131 - Flags: review+
(Assignee)

Comment 12

16 years ago
Comment on attachment 76131 [details] [diff] [review]
Patch without extra crap from other patches.

sr=beard
Attachment #76131 - Flags: superreview+

Comment 13

16 years ago
Checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 14

15 years ago
Chris Petersen is a new QA contact for oji component. His email is:
petersen@netscape.com
Assignee: bnesse → petersen
fixing small error for pmac@netscape.com (filter with : SPAMMAILSUCKS)
Assignee: petersen → beard
QA Contact: pmac → petersen

Comment 16

15 years ago
Verified in the 2002-12-17-08 cfm trunk under OS X 10.2.3.
Status: RESOLVED → VERIFIED

Updated

7 years ago
Component: Java: OJI → Java: OJI
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.