Problem/Motivation

Fix the description of fields on hover in the form element by making it a full sentence. (e.g., The name of launch template → Enter the name of launch template)

The title of fields in the form element should have only the first word capitalized unless the word is usually used that way. (e.g., Max Count → Max count)

Issue fork cloud-3236419

Command icon 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:

About issue forks

Comments

TamakiFujino created an issue. See original summary.

TamakiFujino opened merge request !404

TamakiFujino’s picture

TamakiFujino commented
Status: Active » Needs review
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: Fix the UI/UX of Cloud Service Providers (1) (Form Elements) » Refactor the UI/UX of Cloud Service Providers (1) (Form Elements)
Status: Needs review » Needs work

@tamakifujino

Thank you for the refactoring. I left my comments above. Please fix IAM Role → IAM role, too. Thanks!

TamakiFujino’s picture

TamakiFujino commented
Status: Needs work » Needs review
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Needs review » Needs work

@tamakifujino

Thank you for the update. There are multiple conflicts, so could you please fix them? Also, it is appreciated if you can leave Unresolved as a mark for further review when you finish modifying.

TamakiFujino opened merge request !412

TamakiFujino closed merge request !412

TamakiFujino’s picture

TamakiFujino commented
Status: Needs work » Needs review
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented

@tamakifujino

Thank you for the update. It looks good.

@baldwinlouie

What do you think?

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: Refactor the UI/UX of Cloud Service Providers (1) (Form Elements) » Refactor the UI/UX of AWS Cloud / OpenStack service provider (Form Elements)
Status: Needs review » Reviewed & tested by the community

@tamakifujino

Thank you for the fix. It looks good. I'll merge the patch to 4.x and close this issue as Fixed.

  • yas committed bebe2c6 on 4.x authored by TamakiFujino 
    Issue #3236419 by TamakiFujino, yas: Refactor the UI/UX of AWS Cloud /...

yas closed merge request !404

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.