octfx/mediawiki-extensions-TemplateStylesExtender
1
0
Fork 0
mirror of https://github.com/octfx/mediawiki-extensions-TemplateStylesExtender.git synced 2024-11-15 03:35:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

doc: Update comments

Browse source
This commit is contained in:
Hannes 2021-03-20 09:24:29 +01:00
parent fdabcb7384
commit f5b122aaed
No known key found for this signature in database
GPG key ID: 371029119C5F0D51
2 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
includes/MatcherFactoryExtender.php
View file

@ -40,7 +40,7 @@ class MatcherFactoryExtender extends MatcherFactory {
private static $extendedCssMediaQuery = false;
/**
* This is in reality a complete copy of the parent hook with line 49 and 91 extended
* This is in reality a complete copy of the parent hook with line 68 and 110 extended
* This can very easily break if there is an update upstream
*
* @inheritDoc
@ -160,5 +160,4 @@ class MatcherFactoryExtender extends MatcherFactory {
return $this->cache[$key];
}
}

7
includes/TemplateStylesExtender.php
View file

@ -71,10 +71,11 @@ class TemplateStylesExtender {
] )
);
// This is kind of ugly?
// Match anything*\s?[var anything|anything var]+\s?anything*(!important)?
// The problem is, that var can be used more or less anywhere
// Setting ONLY var as a CssWideKeywordMatcher does not work as we are then limited to one css value
// The problem is, that var() can be used more or less anywhere
// Setting ONLY var as a CssWideKeywordMatcher would limit the matching to one property
// E.g.: color: var( --color-base ); would work
// border: 1px var( --border-type ) black; would not
// So we need to construct a matcher that matches anything + var somewhere
$propertySanitizer->setCssWideKeywordsMatcher(
new Juxtaposition( [