Enhance OVA provider help text fields

[sgratch]

Reference: #646

Enhance and rephrase the help text fields and validations for following OVA dialogs:

1. Create OVA provider,
2. OCP details - URL title icon help text

Changes include supporting formatted help text.

Screenshots

[sgratch]

cc:// @RichardHoch @anarnold97

[yaacov]

help texts should be one line, we have a different issue for multi line help texts #

[sgratch]

help texts should be one line, we have a different issue for multi line help texts #

I wondered about that as well and wanted your feedback.
From user point of view, it's much nicer to have it split into few lines, but I was not sure about the standard.

I will try one line since a tooltip looks overheaded here.

[yaacov]

it's much nicer to have it split into few lines,

You should not have a text that can be split up, if it can be split to two lines it's too long.
The help is not intended to replace documentation, it's a short help text for the input 4 or 5 words, if it's more then that, it's probably not a help text for an input field.

we have a different task, for longer help hints #541

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
9.4% Duplication on New Code

See analysis details on SonarCloud

[RichardHoch]

Maybe "the"?

[sgratch]

@RichardHoch Do you mean replacing "An IP or hostname of.."
with: "The IP or hostname of..." ?

[RichardHoch]

@sgratch Yes, that is my suggestion if in Hebrew you would say "ה" If not, then it's fine as is.

[RichardHoch]

@yaacov @sgratch One small question; otherwise LGTM.
But ... can you send me the text that appears under "Create Provider" ("Create by using...")? I think it needs a little fixing, even though I see there is a space issue.

[sgratch]

@yaacov @sgratch One small question; otherwise LGTM. But ... can you send me the text that appears under "Create Provider" ("Create by using...")? I think it needs a little fixing, even though I see there is a space issue.

@RichardHoch
You refer to this text?:

forklift-console-plugin/packages/forklift-console-plugin/src/modules/Providers/views/create/ProvidersCreatePage.tsx

Line 226 in 57b1085

'Create by using the form or manually entering YAML or JSON definitions, Provider CR stores attributes that enable MTV to connect to and interact with the source and target providers.',

[RichardHoch]

@yaacov @sgratch One small question; otherwise LGTM. But ... can you send me the text that appears under "Create Provider" ("Create by using...")? I think it needs a little fixing, even though I see there is a space issue.

@RichardHoch You refer to this text?:

forklift-console-plugin/packages/forklift-console-plugin/src/modules/Providers/views/create/ProvidersCreatePage.tsx

Line 226 in 57b1085

'Create by using the form or manually entering YAML or JSON definitions, Provider CR stores attributes that enable MTV to connect to and interact with the source and target providers.',

Yes, please.

[sgratch]

@yaacov @sgratch One small question; otherwise LGTM. But ... can you send me the text that appears under "Create Provider" ("Create by using...")? I think it needs a little fixing, even though I see there is a space issue.

@RichardHoch You refer to this text?:

forklift-console-plugin/packages/forklift-console-plugin/src/modules/Providers/views/create/ProvidersCreatePage.tsx

Line 226 in 57b1085

'Create by using the form or manually entering YAML or JSON definitions, Provider CR stores attributes that enable MTV to connect to and interact with the source and target providers.',

Yes, please.

@RichardHoch
The original text is:
Create by using the form or manually entering YAML or JSON definitions, Provider CR stores attributes that enable MTV to connect to and interact with the source and target providers.

This is not part of this PR but will be fixed if needed. What are your comments for this text?

[RichardHoch]

@sgratch "Create Providers by using the form or manually entering YAML or JSON definitions. Provider CRs store attributes that enable MTV to connect to and interact with the source and target providers."