Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reafactor: use AbstractMnemonicsAction to define Menu handlers #983

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

miurahr
Copy link
Member

@miurahr miurahr commented Mar 16, 2024

Pull request type

  • Other (describe below)
    refactor

Which ticket is resolved?

dev-ML https://sourceforge.net/p/omegat/mailman/message/58749145/

What does this PR change?

  • Replace MainWindowMenuHandler#braBrabrahActionperformed method with Action classes that extends with AbstractMnemonicAction class

Other information

Implementation depends on lib-mnemonics enhancement

omegat-org/lib-mnemonics#8

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

@miurahr miurahr marked this pull request as ready for review May 3, 2024 05:27
- refactor exit actions menus

Signed-off-by: Hiroshi Miura <miurahr@linux.com>
@miurahr miurahr force-pushed the topic/miurahr/gui/main/menu-construct-with-properties-abstract-action branch from d0ed739 to e4248fe Compare May 3, 2024 08:35
@miurahr
Copy link
Member Author

miurahr commented May 3, 2024

rebased on current master.

This comment was marked as resolved.

Signed-off-by: Hiroshi Miura <miurahr@linux.com>
@miurahr miurahr force-pushed the topic/miurahr/gui/main/menu-construct-with-properties-abstract-action branch from cb48713 to 80bf094 Compare May 3, 2024 09:20
Signed-off-by: Hiroshi Miura <miurahr@linux.com>
Signed-off-by: Hiroshi Miura <miurahr@linux.com>
@miurahr
Copy link
Member Author

miurahr commented May 3, 2024

Now ready for merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant