Added ability to add target attribute for adminhtml menu items

[Sekiphp]

Description (*)

In OM there is currently now way how to open hyperlinks to new tabs / windows.

Usage

<menu_item translate="title">
    <title>Menu item label</title>
    <action>adminhtml/hello_world/foo</action>
    <target>_blank</target>
</menu_item>

Contribution checklist (*)

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All automated tests passed successfully (all builds are green)
• Add yourself to contributors list

[midlan]

What about framename as target?
https://www.w3schools.com/tags/att_a_target.asp

[Sekiphp]

What about framename as target? https://www.w3schools.com/tags/att_a_target.asp

Until now, noone want it! This feature is added mainly for _blank option, which should be usable for multiple M1 websites. You are nitpicker, Milan! :-)

[midlan]

I am not nitpicker. When adding new feature I just would implement the complete current standard. By your argumentation you should just implement _blank, because you don't need the others 🙃

[Sekiphp]

I am not nitpicker. When adding new feature I just would implement the complete current standard. By your argumentation you should just implement _blank, because you don't need the others 🙃

I'm not convienced about framename support, but I implement it.....

[github-actions]

Unit Test Results

1 files  ±0  1 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
7 runs  ±0  5 ✔️ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit bc33791. ± Comparison against base commit 14afb4b.

[xqiu]

commit bc33791 should not change the default function signature for _buildMenuArray, some plugin may depend on this function's default behavior (e.g. https://github.com/AOEpeople/Clean_SqlReports) .