-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Store properties dialog bounds. #1282
Store properties dialog bounds. #1282
Conversation
Test Results 900 files ±0 900 suites ±0 46m 0s ⏱️ -1s For more details on these failures, see this check. Results for commit ab22a44. ± Comparison against base commit 01ae9f9. ♻️ This comment has been updated with latest results. |
c59bf33
to
27778d9
Compare
27778d9
to
826cd04
Compare
@Override | ||
protected IDialogSettings getDialogBoundsSettings() { | ||
Bundle bundle = WorkbenchPlugin.getDefault().getBundle(); | ||
IDialogSettings settings = PlatformUI.getDialogSettingsProvider(bundle).getDialogSettings(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can't settings also be null here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah no. It look like null is never returned
@Override | ||
protected IDialogSettings getDialogBoundsSettings() { | ||
Bundle bundle = WorkbenchPlugin.getDefault().getBundle(); | ||
IDialogSettings settings = PlatformUI.getDialogSettingsProvider(bundle).getDialogSettings(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah no. It look like null is never returned
826cd04
to
c4864aa
Compare
One of the failing test is unrelated. See: #811 |
Properties dialog is created and shown when 'Properties' action is performed while an object is selected. (e.g. in Project Explorer) Issue eclipse-platform#949
c4864aa
to
ab22a44
Compare
8abe1b1
into
eclipse-platform:master
This reverts commit 8abe1b1. The PreferenceDialog has a custom size calculation that tries to adapt to the current page. That contradicts using the the former size
…eclipse-platform#1895 This reverts commit 8abe1b1. The PreferenceDialog has a custom size calculation that tries to adapt to the current page. That contradicts using the the former size eclipse-platform#1895
…eclipse-platform#1895 This reverts commit 8abe1b1. The PreferenceDialog has a custom size calculation that tries to adapt to the current page. That contradicts using the the former size eclipse-platform#1895
…eclipse-platform#1895 This reverts commit 8abe1b1. The PreferenceDialog has a custom size calculation that tries to adapt to the current page. That contradicts using the the former size eclipse-platform#1895
…eclipse-platform#1895 This reverts commit 8abe1b1. The PreferenceDialog has a custom size calculation that tries to adapt to the current page. That contradicts using the the former size eclipse-platform#1895
Propeties dialog is created and shown when 'Properties' action is performed while an object is selected. (e.g. in Project Explorer)
Issue #949