Comment 1 Remy Suen 2011-09-16 07:46:40 EDT

(In reply to comment #0)
> There is no need for the 'Enable Animations' preference on Windows 7 (and maybe
> other OSes), as there is an OS setting which should be honored.

Dani, that option is also there in 3.x (in the same 'Appearance' preference page). Do you think we should strip it out from there too?

Comment 2 Dani Megert 2011-09-16 07:48:12 EDT

(In reply to comment #1)
> (In reply to comment #0)
> > There is no need for the 'Enable Animations' preference on Windows 7 (and maybe
> > other OSes), as there is an OS setting which should be honored.
> 
> Dani, that option is also there in 3.x (in the same 'Appearance' preference
> page). Do you think we should strip it out from there too?

+1.

Comment 3 Lars Vogel 2019-11-14 03:21:50 EST

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

If the bug is still relevant, please remove the "stalebug" whiteboard tag.

Comment 4 Lars Vogel 2020-05-07 03:22:57 EDT

I agree we should remove the preference and related code. Currently it is only used for maximize and unzoom (see AnimationEngine) and makes the UI jaggy.

Comment 5 Lars Vogel 2020-05-07 03:24:34 EDT

See also Bug 357939, where the feedback was not positive after implementing it and enabling it by default.

Comment 6 Eclipse Genie 2020-05-07 03:31:30 EDT

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

Comment 7 Lars Vogel 2020-05-11 06:28:17 EDT

Dani, Alex, I suggest to remove this bad-performing setting and its support from platform. Please approve or reject.

Comment 8 Alexander Kurtakov 2020-05-11 07:56:46 EDT

I honestly have no idea what is this enabling as I haven't found any visual difference from enabling it.

Comment 9 Lars Vogel 2020-05-11 08:27:56 EDT

(In reply to Alexander Kurtakov from comment #8)
> I honestly have no idea what is this enabling as I haven't found any visual
> difference from enabling it.

If you maximize or and editor the window, you should see a difference.

Comment 10 Lars Vogel 2020-05-12 06:05:30 EDT

(In reply to Alexander Kurtakov from comment #8)
> I honestly have no idea what is this enabling as I haven't found any visual
> difference from enabling it.

So no objections against removing it?

Comment 11 Alexander Kurtakov 2020-05-12 06:14:02 EDT

Not from me. It would be nice if whatever code is responsible for handling this preference change is removed too.

Comment 12 Lars Vogel 2020-05-12 06:16:11 EDT

(In reply to Alexander Kurtakov from comment #11)
> Not from me. It would be nice if whatever code is responsible for handling
> this preference change is removed too.

+1

Comment 13 Eclipse Genie 2020-05-15 23:23:40 EDT

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

Comment 14 Eclipse Genie 2020-05-16 03:34:21 EDT

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

Comment 15 Eclipse Genie 2020-05-19 09:28:44 EDT

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

Comment 16 Eclipse Genie 2020-05-22 06:57:59 EDT

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

Comment 17 Lars Vogel 2020-05-22 06:58:45 EDT

Removed from UI, remaining code will be removed in 4.17