Cómo deshabilitar un botón de forma más elegante

Tengo en una de mis vistas el siguiente código de afeitar:

@if (item.PMApproved != true) {
                    <input type="button" class="btnresetinvoice button" value="Reset" data-invoiceid="@item.InvoiceId" />
                }
                else {
                    <input type="button" class="btnresetinvoice button" value="Reset" data-invoiceid="@item.InvoiceId" disabled="disabled" />
                }

Bastante duro. Básicamente, quiero deshabilitar el botón bajo ciertas condiciones, ya que podrías resolverlo a partir del código. ¿Cuál sería una forma más deseable de hacer esto?

Prueba esto:

<button type="submit" disabled="@(!item.PMApproved)"></button>

No sé qué idioma estás usando, pero es posible que puedas mover tu if declaración más cercana a la diferencia real entre las dos líneas:

<input type="button" class="btnresetinvoice button" value="Reset"
       data-invoiceid="@item.InvoiceId"
       @{ if(item.PMApproved != true) { 
             @:disabled="disabled" 
        } }
/>

Una solución centrada en el marcado con la ayuda de un nuevo método de extensión:

public static class HtmlExtensions
{
   public static HtmlString DisabledIf(this HtmlHelper html, bool condition)
   {
      return new HtmlString(condition ? "disabled=\"disabled\"" : "");
   }
}

En sus puntos de vista, reutilice el wazoo:

<button type="reset" @Html.DisabledIf(someCondition)>Clear Fields</button>

Bien reutilizable, y el marcado renderizado es muy limpio con respecto a los espacios en blanco:

<button type="reset" disabled="disabled">Clear Fields</button>

Usando la maquinilla de afeitar asp.net mvc5:

@if(condition)
{
   <button data-toggle="collapse" data-target="#content">Details</button>
}
else
{
   <button disabled>Details</button>
}

Evita intentar habilitar el botón desde DevTools, porque la maquinilla de afeitar no está visible para DevTools

<input type="button" value="Reset" @{@((!item.PMApproved) ? null : new { disabled = "disabled" })}; />

No es necesario ese código inflado, solo mantenlo simple 🙂

Un ayudante podría ayudar:

public static class HtmlExtensions
{
    public static IHtmlString ApproveButton(this HtmlHelper htmlHelper, MyViewModel item)
    {
        var button = new TagBuilder("input");
        button.Attributes["type"] = "button";
        button.Attributes["value"] = "Reset";
        button.AddCssClass("btnresetinvoice");
        button.AddCssClass("button");
        button.Attributes["data-invoiceid"] = item.InvoiceId.ToString();
        if (item.PMApproved)
        {
            button.Attributes["disabled"] = "disabled";
        }
        return new HtmlString(button.ToString(TagRenderMode.SelfClosing));
    }
}

y luego:

@Html.ApproveButton(item)

<button @(isEnabled ? null : "disabled")>Butt</button>