I noticed there's no uninstall hook so the css we create is left behind. It's not big issue but should be still fixed.

CommentFileSizeAuthor
#2 Remove-high_contrast-on-uninstall.patch578 bytesneograph734

Comments

hkirsman created an issue. See original summary.

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Status: Active » Needs review
StatusFileSize
new Remove-high_contrast-on-uninstall.patch578 bytes

Should be as easy as this? (The code was already in update 8002 for the old path).

Status: Needs review » Needs work

The last submitted patch, 2: Remove-high_contrast-on-uninstall.patch, failed testing. View results

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Status: Needs work » Needs review

  • hkirsman committed 063a7a7 on 8.x-1.x authored by Neograph734 
    Issue #3116364 by Neograph734, hkirsman: Remove high_contrast.css on...
hkirsman’s picture

hkirsman commented
Status: Needs review » Fixed

Tx, I've commited it.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.