Is there a wrapping formatting option for class properties like alignConstants?

BrandonReyes

Am I missing something or the formatter doesn't support it?
My config:

  "php.format.rules.alignConsecutiveAssignments": true,
  "php.format.rules.alignConstants": true,
  "php.format.rules.alignEnumCases": true,
  "php.format.rules.alignMatchArmBodies": true,
  "php.format.rules.arrayInitializersAlignKeyValuePairs": true

Video: https://imgur.com/a/5EH38F0

MiloslavBeno

Hi Brandon,

Thank you for your feedback. We're still missing one more alignment format rule to align properties. This one is on top of our todo list, so it should be out soon.

I think it should also align visibility modifiers, what do you think?

class x {
    var       $v   = 0;
    public    $a   = "aa";
    private   $bb  = "bb";
    protected $ccc = "ccc"; 
}

BrandonReyes

MiloslavBeno Hi, yes, I think it should also align visibility modifiers. Thank you!

MiloslavBeno

This has been finished and it will be in the next pre-release.

Thanks again for bringing this to our attention.

MiloslavBeno

Just a note. There is one issue with the current implementation. Multiple consecutive formats changes the alignment. Will be fixed ASAP.

Thanks for your patience.

MiloslavBeno

Fixed. Will be available soon in the next update.

BrandonReyes

MiloslavBeno Thank you!