Bug 32335 - Eclipse M5, launcher, arguments tab truncated
Summary: Eclipse M5, launcher, arguments tab truncated
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 2.1 RC3   Edit
Assignee: Joe Szurszewski CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 32747 34187 34196 34632 35045 (view as bug list)
Depends on:
Blocks:
 
Reported: 2003-02-20 05:30 EST by Peter Kidson CLA
Modified: 2003-04-07 01:57 EDT (History)
4 users (show)

See Also:


Attachments
Screen shots (124.00 KB, application/msword)
2003-02-20 09:35 EST, Sonia Dimitrov CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Kidson CLA 2003-02-20 05:30:41 EST
Eclipse M5, W2000 sp3.

Compare the bottoms of the arguments tabs in the launchers of the Eclipse M4 
and M5 builds.   In M4, you have 

Radio button	Local directory
Radio button	Workspace
Checkbox	Use default working directory

But in M5, Workspace and Use default working directory are cut off and can 
therefore not be used.

Screen grabs available.  But where do I attach them?  Or send them to?
Comment 1 Sonia Dimitrov CLA 2003-02-20 09:35:30 EST
Created attachment 3595 [details]
Screen shots
Comment 2 Peter Kidson CLA 2003-02-20 12:37:00 EST
Correction:
It happens with M4 as well as M5.  But not with 2.0.2.

The good news is I have nailed it down - it only happens with large fonts.
The way that tab resizes is possibly just assuming (hard-coded??) small fonts.

Windows, Desktop|Properties|Settings|Advanced|Font Size, select either Small 
Fonts or Large Fonts.

It might be worth also looking for where else similar resizing is done.

Regards.


Comment 3 Darin Wright CLA 2003-02-20 15:35:51 EST
I can only cause the problem if I manually re-size the tab to be too short. 
However I can also manually re-size the tab larger again.
Comment 4 Darin Wright CLA 2003-02-21 11:10:18 EST
Via e-mail, "had no idea it was resizable. It is however strange that the 
default layout is too short".

The dialog does re-size to show tabs headers (width), and is supposed to resize 
to height as required as well. However, It does not seem to expand to the 
proper height on the arguments tab.
Comment 5 Jared Burns CLA 2003-02-21 11:21:35 EST
I've seen similar problems on Linux-GTK in recent builds. The launch config
dialog often opens too small to fit all the widgets.
Comment 6 Darin Wright CLA 2003-02-26 14:18:14 EST
It appears the width is re-sizing properly, but the height is not.
Comment 7 Darin Wright CLA 2003-03-01 13:55:31 EST
*** Bug 32747 has been marked as a duplicate of this bug. ***
Comment 8 Joe Szurszewski CLA 2003-03-03 17:29:17 EST
Can duplicate by opening regular launch config dialog (not external tools), 
selecting local Java config type, making dialog very small, then clicking New.  
The Arguments tab is missing most of the working directory widgetry.
Comment 9 Joe Szurszewski CLA 2003-03-05 12:51:56 EST
Fixed in LaunchConfigurationsDialog.resize().  Problem was that resizing was 
being done relative to the previous size of the entire edit area, instead of the 
size of just the old 'client area' of the tabbed folder (probably a result of 
launch config dialog refactoring).  Thus, the new height wasn't accounting for 
the 'Name' field, the spacer, and the height of the tabs in the tabbed folder.  
Also increased the maximum allowable size of the dialog to be 90% of the 
available screen width & height (this had nothing to do with the bug, but just 
seems like a good idea).
Comment 10 Joe Szurszewski CLA 2003-03-05 12:52:10 EST
Please verify, Jared.
Comment 11 Jared Burns CLA 2003-03-05 18:35:47 EST
Verified.
Comment 12 Darin Wright CLA 2003-03-07 11:26:55 EST
The 90% does not seem like a good idea. My launch dialog always opens huge 
(tall) now (since runtime workbench configs have a large source lookup path). I 
keep re-sizing it smaller, but it keeps opening larger than I want. I suggest 
we revert to the 70%.
Comment 13 Peter Kidson CLA 2003-03-07 13:32:28 EST
The (new) user still needs to be made aware that the dialogue is resizable - by 
giving it a  scroll bar maybe, or a different kind of border.  Failing this 
there is no way of knowing that there is more data to be entered to make the 
launch work.  

On my machine the Local_directory, Workspace and Use_default_working_directory 
controls were not visible, and hence I had no idea they existed and needed 
attention.
Comment 14 Darin Wright CLA 2003-03-07 13:50:41 EST
These arguments should be visible with this fix - the problem is now that the 
dialog is too large. We do not intend to add a scroll area.
Comment 15 Darin Wright CLA 2003-03-07 15:56:36 EST
Fell back percentages to 70%.
Comment 16 Darin Wright CLA 2003-03-07 15:56:48 EST
Please verify.
Comment 17 Darin Wright CLA 2003-03-09 20:54:30 EST
*** Bug 34187 has been marked as a duplicate of this bug. ***
Comment 18 Darin Wright CLA 2003-03-09 20:58:29 EST
*** Bug 34196 has been marked as a duplicate of this bug. ***
Comment 19 Darin Wright CLA 2003-03-11 11:03:38 EST
*** Bug 34632 has been marked as a duplicate of this bug. ***
Comment 20 Darin Wright CLA 2003-03-11 22:21:45 EST
I saw my dialog become tall again. Re-opening to ensure we have "all" of the 
fix.
Comment 21 Darin Wright CLA 2003-03-13 12:05:36 EST
The max sixe constants actually size the tab area, rather than the dialog. 
Since there is a dialog surrounding the tab area, these constants are too 
large. For example, making a config with a classpath containing 46 projects 
causes the dialog to grow to the entire height of the screen (since the 
classpath tab requests to be 70% of the height, and the dialog has another 20-
30% of height).

Reduced the constants to 50%. This behaves more friendly - the tab area only 
grows to half the size of the screen in the "worst" cases, and the surrounding 
dialog takes up an extra 20-30%. This is wide enough to display all tabs for 
all configs we have, and looks better.

Fixed.
Comment 22 Darin Wright CLA 2003-03-13 12:05:56 EST
Please verify, Joe.
Comment 23 Darin Wright CLA 2003-03-17 12:03:56 EST
*** Bug 35045 has been marked as a duplicate of this bug. ***
Comment 24 Joe Szurszewski CLA 2003-03-17 17:37:35 EST
Verified.
Comment 25 Peter Kidson CLA 2003-04-07 01:54:46 EDT
Seems fine in RC3b