From: Timo Mueller <mail@timomueller.eu>
To: yocto@yoctoproject.org
Cc: Timo Mueller <timo.mueller@bmw-carit.de>
Subject: [PATCHv3 0/8][eclipse-poky] Add target profile quick switch
Date: Tue, 25 Jun 2013 17:33:38 +0200 [thread overview]
Message-ID: <cover.1372170704.git.timo.mueller@bmw-carit.de> (raw)
From: Timo Mueller <timo.mueller@bmw-carit.de>
Hi,
in conclusion to the previous discussion, we've agreed that the quick
switch menu will enable the user to select only configured profiles.
An unconfigured project specific profile will be shown as a greyed out
option in the menu.
There also has been a bug where the projects specific settings and one
profile could be marked as selected at the same time. I could only
reproduce by naming the target profile "project-specific" which
collides with the radioParamter used for the menu. This case is
resolved in this patch set. I've additionally merged the two places
where the state of the menu was set so have one consistent place.
In any case, I've added one patch marked as TESTING to enable tracing
in the relevant components. In case the problem prevails, please
enable the tracing for th org.yocto.sdk.ide plugin and send me the
output.
Changes in v3:
- Only grey out project specific profile if not configured.
- Fix Bug where multiple menu item could be selected at the same time
Changes in v2:
Handle error when project specific profile is not configure more
gracefully. Instead of showing an error message the button is now
greyed out.
Patches 01..07 contain the implementation with the greyed out menu
button.
After playing around with the two options discussed in the first patch
series, I started to prefer inserting a different button in the menu
allowing the user to quickly navigate to the project preferences
instead of just greying it out. I've added this in the last patch of
the series, so you can compare yourself.
From original cover letter
<snip>
if a user wants to change the used target profile of a project he
currently has to open the project preferences. This can be tedious if
he has to switch the profile often.
This is a small addition which allows the user to quickly switch the
used target profile of a project. Instead of having to open the
project preferences the user can select the project in the navigator
and then choose the desired target profile from a drop-down menu in
the toolbar or from the project menu.
</snip>
01: Small i18n fix
02..04: Refactoring the project specific utils
05..06: Introduce the target profile toolbar switch
07: Adds the target profile switch to the project menu
08: TESTING: Enalbe tracing to pin down the multi-select problem
Best regards,
Timo
Timo Mueller (8):
plugins/sdk.ide: Use an internationalized dialog title
plugins/sdk.ide: Extract project specific util methods
plugins/sdk.ide: Move project specific util methods
plugins/sdk.ide: Remove project context from method names
plugins/sdk.ide: Add command to switch the target profile
plugins/sdk.ide: Add profile switch menu to the toolbar
plugins/sdk.ide: Add profile switch menu to the project menu
TESTING: add tracing for radio menu
plugins/org.yocto.sdk.ide/.options | 2 +
.../OSGI-INF/l10n/bundle.properties | 4 +
plugins/org.yocto.sdk.ide/plugin.xml | 100 ++++++++-
.../sdk/ide/ProjectSpecificContributionItem.java | 69 ++++++
.../sdk/ide/TargetProfileContributionItem.java | 139 ++++++++++++
.../org/yocto/sdk/ide/YoctoSDKMessages.properties | 4 +
.../src/org/yocto/sdk/ide/YoctoUISetting.java | 3 +-
.../sdk/ide/actions/ProfileSwitchHandler.java | 134 ++++++++++++
.../natures/YoctoSDKAutotoolsProjectNature.java | 3 +-
.../ide/preferences/YoctoSDKPreferencePage.java | 11 +-
.../preferences/YoctoSDKProjectPropertyPage.java | 130 ++---------
.../sdk/ide/utils/ProjectPreferenceUtils.java | 240 +++++++++++++++++++++
.../src/org/yocto/sdk/ide/utils/YoctoSDKUtils.java | 118 ----------
.../ide/wizard/NewYoctoProjectTemplateProcess.java | 3 +-
14 files changed, 714 insertions(+), 246 deletions(-)
create mode 100644 plugins/org.yocto.sdk.ide/.options
create mode 100644 plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/ProjectSpecificContributionItem.java
create mode 100644 plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/TargetProfileContributionItem.java
create mode 100644 plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/actions/ProfileSwitchHandler.java
create mode 100644 plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/utils/ProjectPreferenceUtils.java
--
1.8.1.4
next reply other threads:[~2013-06-25 15:33 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-25 15:33 Timo Mueller [this message]
2013-06-25 15:33 ` [PATCHv3 1/8] plugins/sdk.ide: Use an internationalized dialog title Timo Mueller
2013-06-25 15:33 ` [PATCHv3 2/8] plugins/sdk.ide: Extract project specific util methods Timo Mueller
2013-06-25 15:33 ` [PATCHv3 3/8] plugins/sdk.ide: Move " Timo Mueller
2013-06-25 15:33 ` [PATCHv3 4/8] plugins/sdk.ide: Remove project context from method names Timo Mueller
2013-06-25 15:33 ` [PATCHv3 5/8] plugins/sdk.ide: Add command to switch the target profile Timo Mueller
2013-06-25 15:33 ` [PATCHv3 6/8] plugins/sdk.ide: Add profile switch menu to the toolbar Timo Mueller
2013-06-25 15:33 ` [PATCHv3 7/8] plugins/sdk.ide: Add profile switch menu to the project menu Timo Mueller
2013-06-25 15:33 ` [PATCHv3 8/8] TESTING: add tracing for radio menu Timo Mueller
2013-06-25 22:07 ` [PATCHv3 0/8][eclipse-poky] Add target profile quick switch Zhang, Jessica
2013-06-26 7:02 ` Timo Müller
2013-06-26 16:40 ` Zhang, Jessica
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1372170704.git.timo.mueller@bmw-carit.de \
--to=mail@timomueller.eu \
--cc=timo.mueller@bmw-carit.de \
--cc=yocto@yoctoproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.