Commit c40f49bf authored by mathp's avatar mathp Committed by Commit Bot

[Payments] Improve some accessibility labels in editors

BUG=729633
TEST=VoiceOver

Review-Url: https://codereview.chromium.org/2931503002
Cr-Commit-Position: refs/heads/master@{#478600}
parent e9d0b77f
...@@ -283,10 +283,11 @@ CreditCardEditorViewController::CreateCustomFieldView( ...@@ -283,10 +283,11 @@ CreditCardEditorViewController::CreateCustomFieldView(
combobox_layout->StartRow(0, 0); combobox_layout->StartRow(0, 0);
constexpr int kInputFieldHeight = 28; constexpr int kInputFieldHeight = 28;
EditorField tmp_month{autofill::CREDIT_CARD_EXP_MONTH, base::string16(), EditorField tmp_month{
EditorField::LengthHint::HINT_SHORT, autofill::CREDIT_CARD_EXP_MONTH,
/*required=*/true, l10n_util::GetStringUTF16(IDS_AUTOFILL_FIELD_LABEL_EXPIRATION_MONTH),
EditorField::ControlType::COMBOBOX}; EditorField::LengthHint::HINT_SHORT,
/*required=*/true, EditorField::ControlType::COMBOBOX};
std::unique_ptr<ValidatingCombobox> month_combobox = std::unique_ptr<ValidatingCombobox> month_combobox =
CreateComboboxForField(tmp_month); CreateComboboxForField(tmp_month);
*focusable_field = month_combobox.get(); *focusable_field = month_combobox.get();
...@@ -294,9 +295,11 @@ CreditCardEditorViewController::CreateCustomFieldView( ...@@ -294,9 +295,11 @@ CreditCardEditorViewController::CreateCustomFieldView(
views::GridLayout::FILL, views::GridLayout::FILL, views::GridLayout::FILL, views::GridLayout::FILL,
0, kInputFieldHeight); 0, kInputFieldHeight);
EditorField tmp_year{autofill::CREDIT_CARD_EXP_4_DIGIT_YEAR, EditorField tmp_year{
base::string16(), EditorField::LengthHint::HINT_SHORT, autofill::CREDIT_CARD_EXP_4_DIGIT_YEAR,
/*required=*/true, EditorField::ControlType::COMBOBOX}; l10n_util::GetStringUTF16(IDS_AUTOFILL_FIELD_LABEL_EXPIRATION_YEAR),
EditorField::LengthHint::HINT_SHORT,
/*required=*/true, EditorField::ControlType::COMBOBOX};
std::unique_ptr<ValidatingCombobox> year_combobox = std::unique_ptr<ValidatingCombobox> year_combobox =
CreateComboboxForField(tmp_year); CreateComboboxForField(tmp_year);
combobox_layout->AddView(year_combobox.release(), 1, 1, combobox_layout->AddView(year_combobox.release(), 1, 1,
......
...@@ -206,6 +206,7 @@ EditorViewController::CreateComboboxForField(const EditorField& field) { ...@@ -206,6 +206,7 @@ EditorViewController::CreateComboboxForField(const EditorField& field) {
base::MakeUnique<ValidatingCombobox>(GetComboboxModelForType(field.type), base::MakeUnique<ValidatingCombobox>(GetComboboxModelForType(field.type),
CreateValidationDelegate(field)); CreateValidationDelegate(field));
base::string16 initial_value = GetInitialValueForType(field.type); base::string16 initial_value = GetInitialValueForType(field.type);
combobox->SetAccessibleName(field.label);
if (!initial_value.empty()) { if (!initial_value.empty()) {
combobox->SelectValue(initial_value); combobox->SelectValue(initial_value);
combobox->SetInvalid(!combobox->IsValid()); combobox->SetInvalid(!combobox->IsValid());
...@@ -373,6 +374,7 @@ views::View* EditorViewController::CreateInputField(views::GridLayout* layout, ...@@ -373,6 +374,7 @@ views::View* EditorViewController::CreateInputField(views::GridLayout* layout,
// Set the initial value and validity state. // Set the initial value and validity state.
base::string16 initial_value = GetInitialValueForType(field.type); base::string16 initial_value = GetInitialValueForType(field.type);
text_field->SetText(initial_value); text_field->SetText(initial_value);
text_field->SetAccessibleName(field.label);
*valid = text_field->IsValid(); *valid = text_field->IsValid();
if (!initial_value.empty()) if (!initial_value.empty())
text_field->SetInvalid(!(*valid)); text_field->SetInvalid(!(*valid));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment