Bug 457390 - [FastView] Deprecated Fast View API
Summary: [FastView] Deprecated Fast View API
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.5   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.5 M6   Edit
Assignee: Denis Zygann CLA
QA Contact:
URL:
Whiteboard:
Keywords: greatfix
Depends on:
Blocks:
 
Reported: 2015-01-13 18:09 EST by Lars Vogel CLA
Modified: 2015-03-16 19:55 EDT (History)
4 users (show)

See Also:
daniel_megert: pmc_approved-

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 Lars Vogel CLA 2015-01-13 18:09:30 EST 
We do not support fast views for a while, see for example ModeledPageLayout.addFastView which calls E4Util.unsupported("addFastView: " + viewId); //$NON-NLS-1$.

I suggest we remove the related API:

Remove field fastViewWithRadio from org.eclipse.ui.views and the corresponding methods from IPageLayout.
Comment 1 Harald Albers CLA 2015-01-23 13:11:15 EST 
IWorkbenchWindowConfigurer#getShowFastViewBars and setShowFastViewBars should be marked as deprecated
Comment 2 Dani Megert CLA 2015-01-28 11:12:27 EST 
You can mark those things deprecated but not remove them.
Comment 3 Denis Zygann CLA 2015-01-31 16:49:05 EST 
I want to fix the bug. Should I set the API to @Deprecated or remove it?
Comment 4 David Williams CLA 2015-01-31 17:08:52 EST 
(In reply to Denis Zygann from comment #3)
> I want to fix the bug. Should I set the API to @Deprecated or remove it?

@Deprecated

See 
https://wiki.eclipse.org/Eclipse/API_Central/Deprecation_Policy
Comment 5 Denis Zygann CLA 2015-01-31 19:22:41 EST 
My changes https://git.eclipse.org/r/#/c/40833/
Comment 7 Lars Vogel CLA 2015-03-09 07:06:20 EDT 
Thanks Denis for your contribution and sorry for the delay on my side.
Comment 8 Lars Vogel CLA 2015-03-16 19:55:47 EDT 
.