Skip to content

Fix CSS linting errors across project stylesheets#21

Merged
gb119 merged 3 commits into
mainfrom
copilot/fix-css-linting-errors
Mar 3, 2026
Merged

Fix CSS linting errors across project stylesheets#21
gb119 merged 3 commits into
mainfrom
copilot/fix-css-linting-errors

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 3, 2026

Ran stylelint with stylelint-config-standard across project CSS files, resolving all 114 errors.

static/css/labman.css

  • Fixed typo: padding-rightLpadding-right
  • Renamed #btn__nav#btn-nav (kebab-case; updated templates/base.html)
  • Fixed unitless margin: margin: -12 0 0 -12margin: -12px 0 0 -12px
  • Fixed deprecated @media (min-width: Npx)@media (width >= Npx) range notation
  • Modernised colour: rgba(0,0,0,.85)rgb(0 0 0 / 85%)
  • Fixed keyword case: optimizeLegibilityoptimizelegibility
  • Reordered selectors and split combined footer rule to fix no-descending-specificity

apps/autocomplete/static/autocomplete/css/autocomplete.css

  • Renamed all class selectors to kebab-case (e.g. phac_aspc_form_autocompletephac-aspc-form-autocomplete, hasFocushas-focus)
  • Replaced deprecated word-wrap with overflow-wrap and clip: rect(...) with clip-path: inset(50%)
  • Modernised rgba()rgb() with / alpha notation; #eeeeee#eee
  • Fixed shorthand redundancies, currentColorcurrentcolor, bare zero units
  • Reordered selectors to resolve no-descending-specificity violations

Companion updates

All files referencing renamed identifiers updated for consistency:

  • templates/base.htmlbtn__navbtn-nav
  • 4 autocomplete templates + autocomplete.js — updated CSS class references
  • ac_container.html renamed → ac-container.html

Tooling

  • Added .stylelintrc.json (stylelint-config-standard)
  • Added package.json with npm run lint:css script
  • Added node_modules/ to .gitignore

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • labman.leeds.ac.uk
    • Triggering command: /usr/bin/python3 python3 manage.py check (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits March 3, 2026 12:39
Co-authored-by: gb119 <4428426+gb119@users.noreply.github.com>
Co-authored-by: gb119 <4428426+gb119@users.noreply.github.com>
Copilot AI changed the title [WIP] Run stylelint and correct CSS linting errors Fix CSS linting errors across project stylesheets Mar 3, 2026
Copilot finished work on behalf of gb119 March 3, 2026 12:50
@codacy-production
Copy link
Copy Markdown

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00% (target: -1.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (e970a05) 4672 2193 46.94%
Head commit (402aec6) 4672 (+0) 2193 (+0) 46.94% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#21) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@gb119 gb119 marked this pull request as ready for review March 3, 2026 12:56
@gb119 gb119 merged commit 9f5e044 into main Mar 3, 2026
4 checks passed
@gb119 gb119 deleted the copilot/fix-css-linting-errors branch March 3, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants