Skip to content

Replace if (x) { addClass(y) } else { removeClass(y) } with toggleClass(y, x) #1

@KyNorthstar

Description

@KyNorthstar

These could be coalesced into one toggleClass

if (newIsShown) {
defaultElementResponsibleForHidingAndShowingTheSidebar.addClass(sidebarShownClassName)
}
else {
defaultElementResponsibleForHidingAndShowingTheSidebar.removeClass(sidebarShownClassName)
}

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions