Nik

Sounds good. I was hoping to grab @wilr tomorrow and talk about my changes, as they might make for a good PR. But I don't know if there's reasons for the existing layout

Nik

Just trying to get the design to be a bit more... SilverStripe conform, because that's really not what this package is good at 😄

Nik

Interesting... I'll have a dive into that

Nik

haha 🙂 I had a feeling it might be because of the GridField this is in but I can't think of a reason why that should be the case

Nik

Did a build & a flush but it's not giving me what I want

Nik

Here's the original file: https://github.com/silvershop/silverstripe-hasonefield/blob/master/templates/SilverShop/HasOneField/GridFieldHasOneButtonRow.ss

Show 1 attachment(s)
templates/SilverShop/HasOneField/GridFieldHasOneButtonRow.ss

<div class="toolbar--content form-group stacked px-lg-0 field ss-gridfield-buttonrow ss-gridfield-buttonrow-{$TargetFragmentName}">
    <label class="form__field-label">
        $GridField.Title
    </label>
    <div class="form__field-holder form-row">
        <div class="col">$LeftFragment</div>
        <div class="col-auto btn-toolbar">
            $RightFragment
        </div>
    </div>
</div>
Hide attachment content
Nik

This beauty was my try. Unless I'm really dumb right now it should be correct, but it doesn't take it