Bug 394730 - [Xtend/Xpand IDE Integration] Buttons ([New], [Edit..], [Remove],...) must be enable/disable together with enable state of OutletsGroup
Summary: [Xtend/Xpand IDE Integration] Buttons ([New], [Edit..], [Remove],...) must be...
Status: CLOSED WONTFIX
Alias: None
Product: Sphinx
Classification: Automotive
Component: M2x IDE Integration (show other bugs)
Version: 0.7.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-21 00:21 EST by nguyenntk Mising name CLA
Modified: 2024-05-06 23:11 EDT (History)
2 users (show)

See Also:

Attachments
Patch file for OutletsGroup (2.13 KB, application/octet-stream)
2012-11-21 00:21 EST, nguyenntk Mising name CLA 		idydieng: iplog+
Details
View All 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 nguyenntk Mising name CLA 2012-11-21 00:21:49 EST 
Created attachment 223784 [details]
Patch file for OutletsGroup

= Problem description =
Currently, method updateOutletsGroupEnableState(boolean) updates state of table in OutletsGroup only, buttons ([New], [Restore Default],...) in OutletsGroup are not updated. When "Add Xpand/Xtend/Check support" is unchecked, Outlets table is disabled but [New] and [Restore Defaults] buttons are still enable.

It should be enhanced to update state of table in OutletsGroup and these buttons accordingly.


= Resolution proposal =

* Modify method org.eclipse.sphinx.xtendxpand.ui.wizards.pages.OutletsConfigurationPage.updateOutletsGroupEnableState(boolean) to set enable state of restoreDefaultButton

* Modify method org.eclipse.sphinx.xtendxpand.ui.groups.OutletsGroup.setEnabled(boolean) to set enable state of buttons (Add, Edit& Restore Default) according to tableViewer state.

Please view the patch file for more detail!
Comment 1 nguyenntk Mising name CLA 2012-11-21 03:24:43 EST 
And the outletsGroup in OutletsConfigurationPage is a "private", so could you please provide a getter method to access it (as the same outletProvider)
Comment 2 Balazs Grill CLA 2024-05-06 23:11:25 EDT 
Closed stale issue before migration