17348 Enhance shortcut learnability

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

17348 Enhance shortcut learnability

Nicolai Hess-3-2
Can please someone translate / change the text in the
Shortcut reminder Morph settings.( that got integrated in 50521)

see my comment on
https://pharo.fogbugz.com/f/cases/17348/Enhance-shortcut-learnability#BugEvent.149326

and I asked for resubmitting  the changes with proper author initials
instead of "tr" and if he signed the license agreement.
Anyone knows more about "tr" ?

And I am not sure if the right version is integrated.

nicolai
Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

Marcus Denker-4
Hello,

I wil reopen the issue and put your comment there.

Marcus

On 11 Jan 2016, at 09:56, Nicolai Hess <[hidden email]> wrote:

Can please someone translate / change the text in the
Shortcut reminder Morph settings.( that got integrated in 50521)

see my comment on
https://pharo.fogbugz.com/f/cases/17348/Enhance-shortcut-learnability#BugEvent.149326

and I asked for resubmitting  the changes with proper author initials
instead of "tr" and if he signed the license agreement.
Anyone knows more about "tr" ?

And I am not sure if the right version is integrated.

nicolai

Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

Henrik Nergaard

Somehow it integrated

 

______________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.2

Author: tr

Time: 8 January 2016, 4:18:55.135106 pm

UUID: 0aa30f76-84f3-0449-acde-172e56ea3615

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-tr.526

 

better version

 

 

instead of:

 

____________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.2

Author: HenrikNergaard

Time: 8 January 2016, 11:01:18.490743 pm

UUID: 2a6f1c3f-a0b3-524d-9095-02fb42ab8358

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-HenrikNergaard.526

 

Refactor of initial code.

 

I have updated and merged a new slice (in Image 50521) which should fix it. The correct version that should be integrated is:

 

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.5

Author: HenrikNergaard

Time: 11 January 2016, 12:17:08.504781 am

UUID: f0271c17-07a7-0d4e-a6eb-d2fef9b2853c

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.4

Dependencies: Morphic-Base-HenrikNergaard.530

 

remove unused variable.

 

 

 

Best regards,

Henrik

 

 

From: Pharo-dev [mailto:[hidden email]] On Behalf Of Marcus Denker
Sent: Monday, January 11, 2016 11:35 AM
To: Pharo Development List <[hidden email]>
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Hello,

 

I wil reopen the issue and put your comment there.

 

            Marcus

 

On 11 Jan 2016, at 09:56, Nicolai Hess <[hidden email]> wrote:

 

Can please someone translate / change the text in the

Shortcut reminder Morph settings.( that got integrated in 50521)

see my comment on
https://pharo.fogbugz.com/f/cases/17348/Enhance-shortcut-learnability#BugEvent.149326

and I asked for resubmitting  the changes with proper author initials

instead of "tr" and if he signed the license agreement.

Anyone knows more about "tr" ?

And I am not sure if the right version is integrated.

 

nicolai

 

Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

stepharo
Henrik

Thanks. What I would love to get is

    another little morph below the shortcut with the name of the menu
    like that we can learn that the tools xxx is launched with shortcut cmd X

    I think that we should have a preference that does not flush the counters at startup

Stef


Le 11/1/16 12:14, Henrik Nergaard a écrit :

Somehow it integrated

 

______________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.2

Author: tr

Time: 8 January 2016, 4:18:55.135106 pm

UUID: 0aa30f76-84f3-0449-acde-172e56ea3615

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-tr.526

 

better version

 

 

instead of:

 

____________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.2

Author: HenrikNergaard

Time: 8 January 2016, 11:01:18.490743 pm

UUID: 2a6f1c3f-a0b3-524d-9095-02fb42ab8358

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-HenrikNergaard.526

 

Refactor of initial code.

 

I have updated and merged a new slice (in Image 50521) which should fix it. The correct version that should be integrated is:

 

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.5

Author: HenrikNergaard

Time: 11 January 2016, 12:17:08.504781 am

UUID: f0271c17-07a7-0d4e-a6eb-d2fef9b2853c

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.4

Dependencies: Morphic-Base-HenrikNergaard.530

 

remove unused variable.

 

 

 

Best regards,

Henrik

 

 

From: Pharo-dev [[hidden email]] On Behalf Of Marcus Denker
Sent: Monday, January 11, 2016 11:35 AM
To: Pharo Development List [hidden email]
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Hello,

 

I wil reopen the issue and put your comment there.

 

            Marcus

 

On 11 Jan 2016, at 09:56, Nicolai Hess <[hidden email]> wrote:

 

Can please someone translate / change the text in the

Shortcut reminder Morph settings.( that got integrated in 50521)

see my comment on
https://pharo.fogbugz.com/f/cases/17348/Enhance-shortcut-learnability#BugEvent.149326

and I asked for resubmitting  the changes with proper author initials

instead of "tr" and if he signed the license agreement.

Anyone knows more about "tr" ?

And I am not sure if the right version is integrated.

 

nicolai

 


Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

Henrik Nergaard

Hi Stef,

 

Something like this? (see attachment)

 

Best regards,

Henrik

 

From: Pharo-dev [mailto:[hidden email]] On Behalf Of stepharo
Sent: Monday, January 11, 2016 4:52 PM
To:
Pharo Development List <[hidden email]>
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Henrik

Thanks. What I would love to get is

    another little morph below the shortcut with the name of the menu
    like that we can learn that the tools xxx is launched with shortcut cmd X

    I think that we should have a preference that does not flush the counters at startup

Stef

Le 11/1/16 12:14, Henrik Nergaard a écrit :

Somehow it integrated

 

______________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.2

Author: tr

Time: 8 January 2016, 4:18:55.135106 pm

UUID: 0aa30f76-84f3-0449-acde-172e56ea3615

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-tr.526

 

better version

 

 

instead of:

 

____________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.2

Author: HenrikNergaard

Time: 8 January 2016, 11:01:18.490743 pm

UUID: 2a6f1c3f-a0b3-524d-9095-02fb42ab8358

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-HenrikNergaard.526

 

Refactor of initial code.

 

I have updated and merged a new slice (in Image 50521) which should fix it. The correct version that should be integrated is:

 

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.5

Author: HenrikNergaard

Time: 11 January 2016, 12:17:08.504781 am

UUID: f0271c17-07a7-0d4e-a6eb-d2fef9b2853c

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.4

Dependencies: Morphic-Base-HenrikNergaard.530

 

remove unused variable.

 

 

 

Best regards,

Henrik

 

 

From: Pharo-dev [[hidden email]] On Behalf Of Marcus Denker
Sent: Monday, January 11, 2016 11:35 AM
To: Pharo Development List [hidden email]
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Hello,

 

I wil reopen the issue and put your comment there.

 

            Marcus

 

On 11 Jan 2016, at 09:56, Nicolai Hess <[hidden email]> wrote:

 

Can please someone translate / change the text in the

Shortcut reminder Morph settings.( that got integrated in 50521)

see my comment on
https://pharo.fogbugz.com/f/cases/17348/Enhance-shortcut-learnability#BugEvent.149326

and I asked for resubmitting  the changes with proper author initials

instead of "tr" and if he signed the license agreement.

Anyone knows more about "tr" ?

And I am not sure if the right version is integrated.

 

nicolai

 

 


ex2_shortcutReminder.PNG (205K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

stepharo
Sweet!
I was thinking to put it below and smaller but this is cool.

Stef

Le 12/1/16 21:03, Henrik Nergaard a écrit :

Hi Stef,

 

Something like this? (see attachment)

 

Best regards,

Henrik

 

From: Pharo-dev [[hidden email]] On Behalf Of stepharo
Sent: Monday, January 11, 2016 4:52 PM
To:
Pharo Development List [hidden email]
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Henrik

Thanks. What I would love to get is

    another little morph below the shortcut with the name of the menu
    like that we can learn that the tools xxx is launched with shortcut cmd X

    I think that we should have a preference that does not flush the counters at startup

Stef

Le 11/1/16 12:14, Henrik Nergaard a écrit :

Somehow it integrated

 

______________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.2

Author: tr

Time: 8 January 2016, 4:18:55.135106 pm

UUID: 0aa30f76-84f3-0449-acde-172e56ea3615

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-tr.526

 

better version

 

 

instead of:

 

____________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.2

Author: HenrikNergaard

Time: 8 January 2016, 11:01:18.490743 pm

UUID: 2a6f1c3f-a0b3-524d-9095-02fb42ab8358

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-HenrikNergaard.526

 

Refactor of initial code.

 

I have updated and merged a new slice (in Image 50521) which should fix it. The correct version that should be integrated is:

 

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.5

Author: HenrikNergaard

Time: 11 January 2016, 12:17:08.504781 am

UUID: f0271c17-07a7-0d4e-a6eb-d2fef9b2853c

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.4

Dependencies: Morphic-Base-HenrikNergaard.530

 

remove unused variable.

 

 

 

Best regards,

Henrik

 

 

From: Pharo-dev [[hidden email]] On Behalf Of Marcus Denker
Sent: Monday, January 11, 2016 11:35 AM
To: Pharo Development List [hidden email]
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Hello,

 

I wil reopen the issue and put your comment there.

 

            Marcus

 

On 11 Jan 2016, at 09:56, Nicolai Hess <[hidden email]> wrote:

 

Can please someone translate / change the text in the

Shortcut reminder Morph settings.( that got integrated in 50521)

see my comment on
https://pharo.fogbugz.com/f/cases/17348/Enhance-shortcut-learnability#BugEvent.149326

and I asked for resubmitting  the changes with proper author initials

instead of "tr" and if he signed the license agreement.

Anyone knows more about "tr" ?

And I am not sure if the right version is integrated.

 

nicolai

 

 


Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

CyrilFerlicot
In reply to this post by Henrik Nergaard
Le 12/01/2016 21:03, Henrik Nergaard a écrit :

> Hi Stef,
>
>  
>
> Something like this? (see attachment)
>
>  
>
> Best regards,
>
> Henrik
>
>  
Hi,

I think it would be cool to also have a setting to get it with an
#inform: instead of a full Morph.

I would like to get a reminder of the shortcuts but not in all my
screen. A little popup at the bottom left is less invasive.
So I think that we should be able to get both.
What do you think ?


--
Cyril Ferlicot

http://www.synectique.eu

165 Avenue Bretagne
Lille 59000 France


signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

stepharo
ShortcutReminder should handle both
     - get a display strategy
     - it could use growl
     or a morph

Stef

Le 13/1/16 17:03, Cyril Ferlicot D. a écrit :

> Le 12/01/2016 21:03, Henrik Nergaard a écrit :
>> Hi Stef,
>>
>>  
>>
>> Something like this? (see attachment)
>>
>>  
>>
>> Best regards,
>>
>> Henrik
>>
>>  
> Hi,
>
> I think it would be cool to also have a setting to get it with an
> #inform: instead of a full Morph.
>
> I would like to get a reminder of the shortcuts but not in all my
> screen. A little popup at the bottom left is less invasive.
> So I think that we should be able to get both.
> What do you think ?
>
>


Reply | Threaded
Open this post in threaded view
|

Re: 17348 Enhance shortcut learnability

stepharo
In reply to this post by stepharo
Hi henrik

did you publish the code?

Stef
PS: I start playing magic with my third son and he is building fun decks that kill me :)

Le 13/1/16 16:38, stepharo a écrit :
Sweet!
I was thinking to put it below and smaller but this is cool.

Stef

Le 12/1/16 21:03, Henrik Nergaard a écrit :

Hi Stef,

 

Something like this? (see attachment)

 

Best regards,

Henrik

 

From: Pharo-dev [[hidden email]] On Behalf Of stepharo
Sent: Monday, January 11, 2016 4:52 PM
To:
Pharo Development List [hidden email]
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Henrik

Thanks. What I would love to get is

    another little morph below the shortcut with the name of the menu
    like that we can learn that the tools xxx is launched with shortcut cmd X

    I think that we should have a preference that does not flush the counters at startup

Stef

Le 11/1/16 12:14, Henrik Nergaard a écrit :

Somehow it integrated

 

______________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.2

Author: tr

Time: 8 January 2016, 4:18:55.135106 pm

UUID: 0aa30f76-84f3-0449-acde-172e56ea3615

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-tr.526

 

better version

 

 

instead of:

 

____________

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.2

Author: HenrikNergaard

Time: 8 January 2016, 11:01:18.490743 pm

UUID: 2a6f1c3f-a0b3-524d-9095-02fb42ab8358

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-tr.1

Dependencies: Morphic-Base-HenrikNergaard.526

 

Refactor of initial code.

 

I have updated and merged a new slice (in Image 50521) which should fix it. The correct version that should be integrated is:

 

Name: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.5

Author: HenrikNergaard

Time: 11 January 2016, 12:17:08.504781 am

UUID: f0271c17-07a7-0d4e-a6eb-d2fef9b2853c

Ancestors: SLICE-Issue-17348-Enhance-shortcut-learnability-HenrikNergaard.4

Dependencies: Morphic-Base-HenrikNergaard.530

 

remove unused variable.

 

 

 

Best regards,

Henrik

 

 

From: Pharo-dev [[hidden email]] On Behalf Of Marcus Denker
Sent: Monday, January 11, 2016 11:35 AM
To: Pharo Development List [hidden email]
Subject: Re: [Pharo-dev] 17348 Enhance shortcut learnability

 

Hello,

 

I wil reopen the issue and put your comment there.

 

            Marcus

 

On 11 Jan 2016, at 09:56, Nicolai Hess <[hidden email]> wrote:

 

Can please someone translate / change the text in the

Shortcut reminder Morph settings.( that got integrated in 50521)

see my comment on
https://pharo.fogbugz.com/f/cases/17348/Enhance-shortcut-learnability#BugEvent.149326

and I asked for resubmitting  the changes with proper author initials

instead of "tr" and if he signed the license agreement.

Anyone knows more about "tr" ?

And I am not sure if the right version is integrated.

 

nicolai