Skip to content

'no-outline-none' should also check for 'border: none'. #8

@alainEHV

Description

@alainEHV

Right now, this scenario triggers an error/warning for the 'no-outline-none' rule:

&:focus {
    border: transparent;
    outline: none;
}

But these do not trigger an error/warning:

&:focus {
    border: 0;
    outline: none;
}

&:focus {
    border: none;
    outline: none;
}

Looking at the code for the 'no-outline-none' rule, adding an extra check for 'none' should be a small change.
Assuming there's no underlying reason to ignore 'none'.

Some more work might be needed to debug why border: 0 does not trigger any warning.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    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