yellow team mailing list archive

[Matthew Scott <matthew.scott@xxxxxxxxxxxxx>]

Per Ben, subordinate relations can't be removed in Juju. The decision on Friday was to land the branch with the expectation that that functionality would be available in the future, possible in Go Juju. Additionally, it allows the user to keep subordinate relations visible without having to keep their mouse over the indicator. 

~Matthew Scott

PGP key:  http://drab-makyo.com/pgp/

(Sent from my phone, responses may be delayed; forgive any typing errors)

On Nov 11, 2012, at 11:16 AM, Kapil Thangavelu <kapil.thangavelu@xxxxxxxxxxxxx> wrote:

> Subordinate reps can't be removed.
> 
> 
> On Nov 8, 2012, at 11:01 PM, Matthew Scott <matthew.scott@xxxxxxxxxxxxx> wrote:
> 
>> On 2012/11/07 21:57:18, gary.poster wrote:
>>> Hi Matt.  Looks good, thank you.
>> 
>>> I have random UI/UX considerations for you to ponder or ignore.
>> Otherwise it
>>> looks good to land.
>> 
>>> - I wonder about making the active relation number italic in addition
>> to blue,
>>> fwiw.  If that's the desired behavior, cool.
>> 
>>> - I guess it is fine that clicks off the relationship icon function
>> normally and
>>> keep the subordinate relationships drawn.  I kind of expected them to
>> go away.
>>> no big deal.
>> 
>> Although I mentioned there was a branch for cancelling state, it was
>> trivial to pull this bit in in order to propose a nicer branch overall.
>> 
>> 
>>> Thanks again,
>> 
>>> Gary
>> 
>> 
>> 
>> https://codereview.appspot.com/6782063/
>> 
>> --
>> https://code.launchpad.net/~makyo/juju-gui/remove-sub-rels/+merge/131725
>> Your team Juju GUI Hackers is requested to review the proposed merge of lp:~makyo/juju-gui/remove-sub-rels into lp:juju-gui.
> 
> -- 
> https://code.launchpad.net/~makyo/juju-gui/remove-sub-rels/+merge/131725
> You are the owner of lp:~makyo/juju-gui/remove-sub-rels.

-- 
https://code.launchpad.net/~makyo/juju-gui/remove-sub-rels/+merge/131725
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~makyo/juju-gui/remove-sub-rels into lp:juju-gui.