Fixes nextcloud#24450 by making the icon grey only on hover and white otherwise.

RafaOP · RafaOP · commit dff25e02b3b8 · 2021-05-10T18:17:55.000-03:00
diff --git a/apps/theming/css/theming.scss b/apps/theming/css/theming.scss
@@ -92,14 +92,10 @@ $invert: luma($color-primary) > 0.6;
 }
 
 /* Colorized svg images */
-.icon-file, .icon-filetype-text, .icon-file-white:hover {
+.icon-file, .icon-filetype-text, .icon-file-white:hover, .icon-file-white:focus {
 	background-image: url(./img/core/filetypes/text.svg?v=#{$theming-cachebuster});
 }
 
-.icon-file-white, .icon-filetype-text-white {
-	background-image: url(./img/core/filetypes/text-white.svg?v=#{$theming-cachebuster});
-}
-
 .icon-folder, .icon-filetype-folder {
 	background-image: url(./img/core/filetypes/folder.svg?v=#{$theming-cachebuster});
 }
diff --git a/core/css/icons.scss b/core/css/icons.scss
@@ -384,12 +384,15 @@ audio, canvas, embed, iframe, img, input, object, video {
 }
 
 .icon-file,
-.icon-file-white,
-.icon-filetype-text,
 .icon-filetype-text {
 	@include icon-color('text', 'filetypes', #969696, 1, true);
 }
 
+.icon-file-white,
+.icon-filetype-text-white {
+	@include icon-color('text', 'filetypes', $color-white, 1, true);
+}
+
 .icon-filetype-file {
 	@include icon-color('file', 'filetypes', #969696, 1, true);
 }

Original file line number	Diff line number	Diff line change
`@@ -92,14 +92,10 @@ $invert: luma($color-primary) > 0.6;`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`/* Colorized svg images */`
`95`		`-.icon-file, .icon-filetype-text, .icon-file-white:hover {`
	`95`	`+.icon-file, .icon-filetype-text, .icon-file-white:hover, .icon-file-white:focus {`
`96`	`96`	`background-image: url(./img/core/filetypes/text.svg?v=#{$theming-cachebuster});`
`97`	`97`	`}`
`98`	`98`
`99`		`-.icon-file-white, .icon-filetype-text-white {`
`100`		`- background-image: url(./img/core/filetypes/text-white.svg?v=#{$theming-cachebuster});`
`101`		`-}`
`102`		`-`
`103`	`99`	`.icon-folder, .icon-filetype-folder {`
`104`	`100`	`background-image: url(./img/core/filetypes/folder.svg?v=#{$theming-cachebuster});`
`105`	`101`	`}`
Original file line number	Diff line number	Diff line change
`@@ -384,12 +384,15 @@ audio, canvas, embed, iframe, img, input, object, video {`
`384`	`384`	`}`
`385`	`385`
`386`	`386`	`.icon-file,`
`387`		`-.icon-file-white,`
`388`		`-.icon-filetype-text,`
`389`	`387`	`.icon-filetype-text {`
`390`	`388`	`@include icon-color('text', 'filetypes', #969696, 1, true);`
`391`	`389`	`}`
`392`	`390`
	`391`	`+.icon-file-white,`
	`392`	`+.icon-filetype-text-white {`
	`393`	`+ @include icon-color('text', 'filetypes', $color-white, 1, true);`
	`394`	`+}`
	`395`	`+`
`393`	`396`	`.icon-filetype-file {`
`394`	`397`	`@include icon-color('file', 'filetypes', #969696, 1, true);`
`395`	`398`	`}`