This is currently a text field. Let's improve the UX by making the "Default currency" element a select list instead.

Patch forthcoming.

CommentFileSizeAuthor
#5 make_the_default-2745105-5.patch1.37 KBmarkdorison
#2 recurly-convert_default_currency_from_textfield_to_select-2745105-2.patch1.63 KBcolan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

colan created an issue. See original summary.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Status: Active » Needs review
FileSize
recurly-convert_default_currency_from_textfield_to_select-2745105-2.patch1.63 KB

See attached.

Ideally, we'd show the more human-readable list from the list of supported currencies, but that can be done in a separate issue.

  • markdorison committed 4f1b485 on 8.x-1.x authored by colan 
    Issue #2745105 by colan: Make the default currency a select list on the...
markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Version: 8.x-1.x-dev » 7.x-2.x-dev
Assigned: colan » Unassigned
Status: Needs review » Patch (to be ported)
markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Status: Patch (to be ported) » Needs review
Issue tags: +needs backport to 7.x-2.x, +needs backport to 7.x-1.x
FileSize
make_the_default-2745105-5.patch1.37 KB
markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Issue tags: -needs backport to 7.x-1.x
markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Version: 7.x-2.x-dev » 7.x-3.x-dev

  • markdorison committed c10b252 on 7.x-3.x authored by colan 
    Issue #2745105 by markdorison, colan: Make the default currency a select...
markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Status: Needs review » Fixed
Issue tags: -needs backport to 7.x-2.x

Status: Fixed » Closed (fixed)

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