Faculty of Information Technology
Software Engineering Group

Opened 6 years ago

Closed 5 years ago

#169 closed defect (fixed)

When changing a shortcut key for button send it should show the current shortcut

Reported by: Patrik Svestka Owned by:
Priority: minor Milestone: 8.0.0
Component: default Keywords:
Cc: Also affects CVS HEAD (eXept version): no

Description


Attachments (2)

libtool_fix_1_of_1_rev_1334d79e9cf4__169_When_changing_a_shortcut_key_for_button_send_it_should_show_the_current_shortcut.patch​ (1.1 KB) - added by Patrik Svestka 6 years ago.
Fixing the Send button shortcut display
libtool_fix_1_of_1_rev_60274b66bfb4__169_When_changing_a_shortcut_key_for_all_shown_buttons_by_default___the_buttons___menu_it_should_show_the_current_shortcut.patch​ (5.0 KB) - added by Patrik Svestka 6 years ago.
All shown buttons can be now mapped at the debugger window

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by Patrik Svestka

Will post a patch but this will help in DebugView?.st :)

initializeStepButtonIn:bpanel
    stepButton := Button
                label:(resources string:'Debug_Step') , (self shortKeyStringForAction: #DebuggerStep)
                action:[
                    stepButton turnOff.
                    self doStep
                ]
                in:bpanel.

    stepButton name:'stepButton'.
!

Changed 6 years ago by Patrik Svestka

Attachment: libtool_fix_1_of_1_rev_1334d79e9cf4__169_When_changing_a_shortcut_key_for_button_send_it_should_show_the_current_shortcut.patch​ added

Fixing the Send button shortcut display

comment:2 Changed 6 years ago by Patrik Svestka

Status: new → testing

comment:3 Changed 6 years ago by Jan Vrany

Patch looks good. Could you extend it to cover all other buttons shown by default? In the future, may like to assign shortcuts to them as well. Thanks!

comment:4 Changed 6 years ago by Jan Vrany

Status: testing → needs_work

comment:5 Changed 6 years ago by Patrik Svestka

Now I'm attaching a patch for shortcut mapping to all shown, by default, buttons at the debugger. Should I delete the older patch?

Changed 6 years ago by Patrik Svestka

Attachment: libtool_fix_1_of_1_rev_60274b66bfb4__169_When_changing_a_shortcut_key_for_all_shown_buttons_by_default___the_buttons___menu_it_should_show_the_current_shortcut.patch​ added

All shown buttons can be now mapped at the debugger window

comment:6 Changed 6 years ago by Patrik Svestka

Status: needs_work → testing

comment:7 Changed 5 years ago by Jan Vrany

Milestone: → 8.0.0
Resolution: → fixed
Status: testing → closed

Fixed a long time ago in:

Note: See TracTickets for help on using tickets.