Help menu form conversion

[liu-samuel]

Fixes: #7397

Before:

After:

[miq-bot]

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).

[miq-bot]

Checked commits liu-samuel/manageiq-ui-classic@d93588a~...872b1e9 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
4 files checked, 18 offenses detected

app/views/ops/_settings_help_menu_tab.html.haml

• ⚠️ - Line 10 - Line contains trailing whitespace
• ⚠️ - Line 11 - Layout/ArgumentAlignment: Align the arguments of a method call if they span more than one line.
• ⚠️ - Line 11 - Layout/HashAlignment: Align the keys and values of a hash literal if they span more than one line.
• ⚠️ - Line 4 - 5 consecutive Ruby scripts can be merged into a single :ruby filter
• ⚠️ - Line 4 - Avoid using instance variables in partials views
• ⚠️ - Line 4 - Line is too long. [86/80]
• ⚠️ - Line 4 - Lint/AmbiguousOperatorPrecedence: Wrap expressions with varying precedence with parentheses to avoid ambiguity.
• ⚠️ - Line 5 - 4 consecutive Ruby scripts can be merged into a single :ruby filter
• ⚠️ - Line 5 - Avoid using instance variables in partials views
• ⚠️ - Line 5 - Line is too long. [90/80]
• ⚠️ - Line 5 - Lint/AmbiguousOperatorPrecedence: Wrap expressions with varying precedence with parentheses to avoid ambiguity.
• ⚠️ - Line 6 - 3 consecutive Ruby scripts can be merged into a single :ruby filter
• ⚠️ - Line 6 - Avoid using instance variables in partials views
• ⚠️ - Line 6 - Line is too long. [87/80]
• ⚠️ - Line 6 - Lint/AmbiguousOperatorPrecedence: Wrap expressions with varying precedence with parentheses to avoid ambiguity.
• ⚠️ - Line 7 - Style/HashSyntax: Use hash rockets syntax.
• ⚠️ - Line 7 - Style/HashSyntax: Use hash rockets syntax.
• ⚠️ - Line 7 - Style/HashSyntax: Use hash rockets syntax.

[miq-bot]

Some comments on commits liu-samuel/manageiq-ui-classic@d93588a~...872b1e9

498c9cc

• ⚠️ - Merge commit 498c9cc detected. Consider rebasing.

[miq-bot]

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).

[GilbertCherrie]

Issues with this pr

• The help form is appearing on other tabs of the settings
• The old form submit button is still present

Need to fix these before merge.

[GilbertCherrie]

Need to fix the issues listed above first