Bugzilla – Full Text Bug Listing
|
Description
Andreas Sewe
(In reply to Andreas Sewe from comment #0) > Hi, > > when sanity checking all views of my Neon M5 installation as part of testing > the Eclipse IDE for Java Developers EPP package, I stumbled upon the > "Classic Search" view. Is this still needed? (I have been using Eclipse for > year, as never seen it before.) > > AFAICT, I can't even start a search whose results will be displayed there > instead of in the modern "Search" view. In particular, the "Search" button > in the "Classic Search" view *itself* starts a search whose results are > shown in the "Search" view. All old plug-ins that offer search and did not migrate would stop working. (In reply to Dani Megert from comment #1) >> AFAICT, I can't even start a search whose results will be displayed there >> instead of in the modern "Search" view. In particular, the "Search" button >> in the "Classic Search" view *itself* starts a search whose results are >> shown in the "Search" view. > > All old plug-ins that offer search and did not migrate would stop working. Fair enough. AFAICT, it has been deprecated since 3.0, so I was just wondering if there were any plans to remove it in the future. Apparently not. It's time to reconsider this IMHO. (In reply to Alexander Kurtakov from comment #3) > It's time to reconsider this IMHO. +1 As this has been deprecated for "software ages" I'm going to remove for 4.22 M2. (In reply to Alexander Kurtakov from comment #5) > As this has been deprecated for "software ages" I'm going to remove for 4.22 > M2. I have no idea what that is and if that is used by anyone, but please announce this on cross project list, with info what exactly is removed (API, view id, possible extension points etc). The people should know where to look for possible breakage. New Gerrit change created: https://git.eclipse.org/r/c/platform/eclipse.platform.text/+/186120 New Gerrit change created: https://git.eclipse.org/r/c/platform/eclipse.platform.text/+/186121 (In reply to Andrey Loskutov from comment #6) > (In reply to Alexander Kurtakov from comment #5) > > As this has been deprecated for "software ages" I'm going to remove for 4.22 > > M2. > > I have no idea what that is and if that is used by anyone, but please > announce this on cross project list, with info what exactly is removed (API, > view id, possible extension points etc). The people should know where to > look for possible breakage. Done. Gerrit change https://git.eclipse.org/r/c/platform/eclipse.platform.text/+/186120 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=bca70eaca7f2f90cdb07d5ae549fe28e5cc0d6f4 Gerrit change https://git.eclipse.org/r/c/platform/eclipse.platform.text/+/186121 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=c08b0fe8204636808e25ea5b88e6a9ce218d619d New Gerrit change created: https://git.eclipse.org/r/c/platform/eclipse.platform.text/+/186526 New Gerrit change created: https://git.eclipse.org/r/c/platform/eclipse.platform.common/+/186527 Gerrit change https://git.eclipse.org/r/c/platform/eclipse.platform.text/+/186526 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=d7ee493d0cb6496fd0e1b915765ec05b2479b83a Gerrit change https://git.eclipse.org/r/c/platform/eclipse.platform.common/+/186527 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.common.git/commit/?id=964d0a51b8d498ce69e221ce481cb4391658fa50 Alex, is this done? |