Possibility to override the label in country widget

[mablae]

By passing in a closure into the form options it's possible to customize the way the label of the country field is formatted:

[xabbuh]

Imo you should then also set the allowed types for this option to string and callable.

[mablae]

@xabbuh Thanks for your feedback. I wasn't even sure if this is the right way to do it?

I will check the required types thing you mentioned

[mablae]

@xabbuh Is a default sprintf() call okay, in order to get rid off the else branch?

[xabbuh]

Where would you place that sprintf() call?

[thoroc]

What happened with this PR? can it be merged?

[mablae]

@thoroc I forgot it to be honest. I will look into it tonight.

Thanks for the reminder

[thoroc]

@mablae @xabbuh @thewilkybarkid any chances of merging this?

[thewilkybarkid]

Sorry for the delay in looking at this. We'll need some test coverage before merging.

[shakaran]

@xabbuh @thewilkybarkid BUMP Please check this issue for merge as soon is possible, it is a nice feature could customize the label or even the class (the test coverage should be pretty simple)

[shakaran]

I just see to that the template https://github.com/misd-service-development/phone-number-bundle/blob/master/Resources/views/Form/tel_bootstrap.html.twig is not picket the default merge params in composed widget that should be useful.

For example in twig I have to hard customize as following via form.phone.country and via form.phone.number:

<div class="row-fluid inline" style="display:inline-block">
	<div class="inline 4u" style="display:inline-block">
	{{ form_widget(form.phone.country, {'attr' : { 'class' : 'control-label inline no-padding-right' } }) }}
	</div>
	
	<div class="inline 7u" style="display:inline-block">
	{{ form_widget(form.phone.number, {'attr' : { 'style': 'display:inline-block', 'class' : '10u col-sm-9 control-label inline no-padding-right' } }) }}
	</div>
</div>

[yellow1912]

Any update on this? Seem to have a conflict in the PhoneNumberType?

[myfbone]

Smth new about this issue?

[mablae]

Maybe we should finally write the tests?

[jimiero]

@mablae @xabbuh @thewilkybarkid any update with this? would be nice to have it merged

[kironet]

Any news?