Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Remove the negative expression from the checkbox description that appears on hover, to avoid the user’s confusion.
Issue fork cloud-3236437
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3236437-fix-the-uiux changes, plain diff MR !405
Comments
Comment #3
mohit.bansal623 CreditAttribution: mohit.bansal623 as a volunteer and at Axelerant commentedAs the merge request is already created, moving it to review.
Comment #4
yas@tamakifujino
Thank you for refactoring. I posted my comment above. Please check. Thanks!
Comment #5
mohit.bansal623 CreditAttribution: mohit.bansal623 as a volunteer and at Axelerant commentedIt looks good.
Comment #6
TamakiFujino CreditAttribution: TamakiFujino commented@yas
To me
Test only when checked
sounds not a full sentence because 'test only' is probably a noun in this case.- Test-only / when checked.
If we use 'test' as verb,
- Test / only when checked. It will avoid a new instance from launching for testing.
However, in this case, it is unclear what to test. If the checkbox means the form will be test version. you probably want to say:
Use this AWS Cloud as test (version/mode). It will avoid a new instance from launching.
Although I am not 100% confident with my grammar and the purpose of the checkbox, I think it is still unclear when we say
test only when checked
Please let me know what the checkbox supposed to do, or any opinion on this.
Comment #7
yas@tamakifujino
Thank you for your suggestion. I posted my comment, what about this one?
'Enable test mode when checked. It will avoid a new instance from launching for testing.'
Comment #8
yas@tamakifujino
As we discussed, I'll merge the patch to
4.x
and close this issue as Fixed.Comment #11
yas