Problem/Motivation

Align the length after the decimal point on CostsBlocks.

CommentFileSizeAuthor
#9 hotfix-3207945-9.patch1.65 KBmasami
#5 3207945-5.patch20.53 KBmasami
#4 3207945-4.patch12.84 KBmasami
#3 3207945.patch12.39 KBmasami

Issue fork cloud-3207945

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

Masami created an issue. See original summary.

Masami opened merge request !206

masami’s picture

masami commented
Status: Active » Needs review
StatusFileSize
new 3207945.patch12.39 KB
masami’s picture

masami commented
StatusFileSize
new 3207945-4.patch12.84 KB
masami’s picture

masami commented
StatusFileSize
new 3207945-5.patch20.53 KB
yas’s picture

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

@masami

Thank you for the refactoring. I tested the patch and it looks great. I like it. I'll merge the patch to 3.x and close this issue as Fixed.

  • yas committed 21092cd on 3.x authored by Masami 
    Issue #3207945 by Masami, yas: Align the length after the decimal point...
yas’s picture

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

@masami

As per our offline discussion, let us change to the status Needs work.

masami’s picture

masami commented
Status: Needs work » Needs review
StatusFileSize
new hotfix-3207945-9.patch1.65 KB
yas’s picture

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

@masami

Thank you for fixing the issue. I tested the patch and it looks good now. I'll merge the patch to 3.x and close this issue as Fixed.

  • yas committed 5ee546e on 3.x authored by Masami 
    Issue #3207945 by Masami, yas: Hotfix - Align the length after the...

yas closed merge request !206

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.