Skip to content

Commit 98d4447

Browse files
committed
:octocat: removed the fill attributes from QRMarkupSVG::path() as they can be set by CSS and were unintuitive
1 parent b8cebb6 commit 98d4447

File tree

1 file changed

+1
-13
lines changed

1 file changed

+1
-13
lines changed

src/Output/QRMarkupSVG.php

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -145,19 +145,7 @@ protected function paths():string{
145145
* @see https://developer.mozilla.org/en-US/docs/Web/SVG/Element/path
146146
*/
147147
protected function path(string $path, int $M_TYPE):string{
148-
$val = $this->getModuleValue($M_TYPE);
149-
// ignore non-existent module values
150-
$format = empty($val)
151-
? '<path class="%1$s" d="%2$s"/>'
152-
: '<path class="%1$s" fill="%3$s" fill-opacity="%4$s" d="%2$s"/>';
153-
154-
return sprintf(
155-
$format,
156-
$this->getCssClass($M_TYPE),
157-
$path,
158-
($val ?? ''), // value may or may not exist
159-
$this->options->svgOpacity
160-
);
148+
return sprintf('<path class="%s" d="%s"/>', $this->getCssClass($M_TYPE), $path);
161149
}
162150

163151
/**

0 commit comments

Comments
 (0)