]> git.koha-community.org Git - koha.git/commit
Bug 34392: Run automated Stylelint fixes on staff CSS
authorOwen Leonard <oleonard@myacpl.org>
Tue, 19 Sep 2023 13:37:20 +0000 (13:37 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Wed, 18 Oct 2023 19:00:34 +0000 (16:00 -0300)
commitb680695142cc4b1b7bfc95e6786f88118676214d
treef30ef7ff11f8513b7bb6f973d1714ed37fc9866e
parent09459e3dd5e428e4c4ed00b6e784cab8cbebbab1
Bug 34392: Run automated Stylelint fixes on staff CSS

This patch corrects errors raised by the new stylelint configuration in
staff-global.scss. Many of the fixes were made automatically by using
the "--fix" option:

npx stylelint koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
--fix

Some examples of errors found:

    - Expected single space after ":"
    - Expected single space after "("
    - Expected single space before ")"
    - Expected empty line before comment
    - Expected no more than 1 empty line
    - Expected a trailing semicolon

Manual fixes include replacing named colors with hex codes, combining
duplicate selectors, removing empty blocks, etc.

Some warnings remain: "Unexpected !important" requires more than a
quick fix, if we should choose to address them.

To test, apply the patch and rebuild the staff interface CSS:
https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client

The changes affect CSS all around the staff interface, but there should
be no visible results of the patch. Everything should look correct.

The stylelint check should return only warnings about "!important"

npx stylelint koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss

Signed-off-by: Sam Lau <samalau@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss