[yocto] [PATCHv2 0/8][eclipse-poky] Add target profile quick switch

[Zhang, Jessica]

Hi Timo,

For case 1, I think we probably should bring back the grey out option.  In my opinion, it's more consistent.  The property window should be the centralized place for user to setup the settings.  The drop down list is for user to quickly toggle between the different settings.  So the behavior should be:

If there's no project specific settings, that option will be greyed out.  I don't think it'll make it secondary as long as it's still visible.  Once user 1st time check the project specific setting in the project properties window, we copy the prior profile settings as start point then the user can further customize if there's the need.  Once a project has its own specific settings, the option will be enabled in the drop down list and the selection will be consistent between the drop down list selection and project properties settings.

What do you think?

Jessica
-----Original Message-----
From: Timo Mueller [mailto:mail at timomueller.eu]
Sent: Sunday, June 23, 2013 1:36 AM
To: Zhang, Jessica
Cc: yocto at yoctoproject.org; Timo Mueller
Subject: Re: [yocto] [PATCHv2 0/8][eclipse-poky] Add target profile quick switch

Hi Jessica,

Am 21.06.2013 23:47, schrieb Zhang, Jessica:
> Hi Timo,
 >
 > So what's the purpose of "Project Specific Settings", my  > understanding is it's something that only local to project instead of  > global name space, e.g. profile?

Yes, that's the main idea.

>  OK, I've played a little bit  of
 > toggling between profiles, and project specific setting and notice  > the following behaviors that can be confusing:
 >
 > 1. As your preference that user can specify "project specific  > settings".  I've noticed for this case, if I haven't entered any  > project specific settings via the preference wizard, it'll always  > copy the profile settings prior to its selection, then in this case I  > don't think it makes much sense since we already have the profiles to  > cover all the settings

That's independent of this patch set, right?

The original idea behind copying the settings of the currently selected profile was to give the user a starting point for creating the project specific settings. The assumption was that the user most of the times only does small modifications to an existing profile.

Sure, if the user doesn't alter the settings, the project specific ones match an already defined target profile. One difference remains, if he changes the target profile it will update all projects that use this profile. The project that is using it's own configuration won't be changed, although the settings are similar.

An alternative to copying the inital settings would be to blank out the form, when the checkbox is checked. If the assumption is not correct and we think that this is easier to understand for the user, I'll provide a patch set changing this behaviour.

>
 > 2.  OK, now let's move to the case of that I do have explicit setup a  > project specific settings in the project property view apply it and  > exit the window.  After that, in the drop down list, when I toggle  > between the options, e.g. I have two profiles besides the project  > specific settings, then every time after I made the selection and  > check in the project properties view, the settings matches my  > selection.  So this is good and consistent behavior.
 >
 > 3. If  inside the project properties window, I uncheck the "Use  > project specific settings" box, and select a profile then apply the  > changes close the window.  Now in the drop down list I will see both  > the "project specific setting" and the selected profile are selected.
 > Then next time when I go back, and re-check "Use project specific  > settings" box, my project specific setting will be gone, instead the  > settings will show as my previous profile settings.

That's a bug in my implementation. They should always be in sink as you explained in your second point.

>
 > Overall, I think we still need to play a little bit more of different  > setting selections via different approaches, some of the inconsistent  > behavior can get user lost.
 >
 > Thanks, Jessica -----Original Message----- From:
 > yocto-bounces at yoctoproject.org
 > [mailto:yocto-bounces at yoctoproject.org] On Behalf Of Timo Mueller  > Sent: Friday, June 21, 2013 5:45 AM To: yocto at yoctoproject.org Cc:
 > Timo Mueller Subject: [yocto] [PATCHv2 0/8][eclipse-poky] Add target  > profile quick switch  >  > From: Timo Mueller <timo.mueller at bmw-carit.de>  >  > Changes in v2: Handle error when project specific profile is not  > configured 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: Experimental: Add  > button to open project preferences  >  > Best regards, Timo

Thanks, for the input. I'll fix the bug and send a v3.

Best regards,
Timo