Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 537998

Summary:	Prompt user to restart Eclipse SDK on Zoom change notification
Product:	[Eclipse Project] Platform	Reporter:	Niraj Modi <niraj.modi>
Component:	UI	Assignee:	Niraj Modi <niraj.modi>
Status:	VERIFIED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	cdtdoug, daniel_megert, kalyan_prasad, lshanmug
Version:	4.9
Target Milestone:	4.9 M3
Hardware:	PC
OS:	Windows 10
See Also:	https://git.eclipse.org/r/127507 https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=3e7a9c5a436ad1b0897884d12c0cbb27fed0ff30
Whiteboard:
Bug Depends on:	515502
Bug Blocks:	534820, 534932, 539830

Description Niraj Modi

2018-08-16 10:01:43 EDT

On Win10 with 'dpiAwareness' setting enabled(https://bugs.eclipse.org/bugs/show_bug.cgi?id=533135#c5) SWT now throws zoomChanged event(via bug 515502)

We should prompt user to restart Eclipse SDK on zommChanged notification.

Comment 1 Niraj Modi

2018-08-16 10:06:51 EDT

Am working on the changes, will share a gerrit shortly.

Comment 2 Eclipse Genie

2018-08-16 10:22:12 EDT

New Gerrit change created: https://git.eclipse.org/r/127507

Comment 3 Doug Schaefer

2018-08-16 11:53:33 EDT

So when I drag a window from one monitor with a certain DPI to another monitor with a different DPI, it's going to prompt me to restart?

Comment 4 Niraj Modi

2018-08-17 06:32:54 EDT

(In reply to Doug Schaefer from comment #3)
> So when I drag a window from one monitor with a certain DPI to another
> monitor with a different DPI, it's going to prompt me to restart?

Current support is only for single monitor zoom(DPI) change scenario.
We will plan to support Multi-monitor thing at later point in time.

Comment 5 Eclipse Genie

2018-08-17 07:50:22 EDT

Gerrit change https://git.eclipse.org/r/127507 was merged to [master].
Commit: http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=3e7a9c5a436ad1b0897884d12c0cbb27fed0ff30

Comment 6 Doug Schaefer

2018-08-17 11:14:39 EDT

(In reply to Niraj Modi from comment #4)
> (In reply to Doug Schaefer from comment #3)
> > So when I drag a window from one monitor with a certain DPI to another
> > monitor with a different DPI, it's going to prompt me to restart?
> 
> Current support is only for single monitor zoom(DPI) change scenario.
> We will plan to support Multi-monitor thing at later point in time.

So are you saying that when I drag my window to a different monitor with a different DPI it won't prompt me to restart?

Comment 7 Niraj Modi

2018-08-21 03:04:38 EDT

Marking resolved, will be adding N&N shortly.

Comment 8 Niraj Modi

2018-08-23 04:41:22 EDT

N&N contents added via below git commit:
http://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=40ddd11a730934555459888579f412cc94400025

Verified fix on Win10 using Build id: I20180820-2000