Bug 493216 - Preferences dialog should remember size
Summary: Preferences dialog should remember size
Status: CLOSED DUPLICATE of bug 98130
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.5.2   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2016-05-09 05:14 EDT by Björn Michael CLA
Modified: 2016-05-11 10:40 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Björn Michael CLA 2016-05-09 05:14:29 EDT 
Each time I open Preferences dialog from Windows menu I have to resize the dialog larger manually because almost always the available space on monitor isn't used for presenting content.
As I close and open the Preferences dialog again the content is sqeezed in a little window and have to be resized.
Comment 1 Lars Vogel CLA 2016-05-09 07:41:34 EDT 
Gerrit reviews welcome
Comment 2 Patrik Suzzi CLA 2016-05-11 06:57:02 EDT 
I took a look at WorkbenchPreferenceDialog#getDialogBoundsStrategy(), and it it is enough to use DIALOG_PERSISTSIZE | DIALOG_PERSISTLOCATION.
Comment 3 Eclipse Genie CLA 2016-05-11 07:01:59 EDT 
New Gerrit change created: https://git.eclipse.org/r/72491
Comment 4 Dani Megert CLA 2016-05-11 08:59:55 EDT 

*** This bug has been marked as a duplicate of bug 98130 ***
Comment 5 Björn Michael CLA 2016-05-11 09:47:09 EDT 
Thanks Patrik but there are other opinions out there.
Comment 6 Patrik Suzzi CLA 2016-05-11 10:40:06 EDT 
(In reply to Björn  Michael from comment #5)
> Thanks Patrik but there are other opinions out there.

NP, I was aware of this as per comments ;) 

/**
* Overridden to persist only the location, not the size, since the current
* page dictates the most appropriate size for the dialog.
*
* @since 3.2
*/
@Override
protected int getDialogBoundsStrategy() {
  return DIALOG_PERSISTLOCATION;
}