Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

currentColor is not converted when importing SVG #2276

Closed
cyberalien opened this issue Sep 16, 2022 · 1 comment
Closed

currentColor is not converted when importing SVG #2276

cyberalien opened this issue Sep 16, 2022 · 1 comment
Labels
bug managed on taiga This issue has been moved to our project at Taiga.io
Milestone

Comments

@cyberalien
Copy link

To Reproduce

Steps to reproduce the behavior:

  1. Import this SVG:
<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" preserveAspectRatio="xMidYMid meet" viewBox="0 0 24 24"><path fill="currentColor" d="m9 20l-1.4-1.4l1.75-1.8q-3.2-.425-5.275-1.75T2 12q0-2.075 2.888-3.538Q7.775 7 12 7t7.113 1.462Q22 9.925 22 12q0 1.55-1.663 2.775Q18.675 16 16 16.6v-2.05q1.925-.5 2.962-1.238Q20 12.575 20 12q0-.8-2.137-1.9Q15.725 9 12 9q-3.725 0-5.862 1.1Q4 11.2 4 12q0 .6 1.275 1.438Q6.55 14.275 8.9 14.7l-1.3-1.3L9 12l4 4Z"/></svg>
  1. Select imported path

Expected behavior

Expected to to see a shape with fill.

Actual behavior

  • Fill is missing
  • In Imported SVG Attributes section fill is set to currentColor.

Screenshots

Screenshot 2022-09-16 at 23 23 25

Desktop (please complete the following information):

  • OS (e.g. iOS): macOS 12.5.1
  • Browser & version (e.g. Chrome 89.0): Chrome 105.0

Environment (please complete the following information):

Additional context:

All editors replace currentColor with black color, so I was expecting penpot to behave like that too.

@cyberalien cyberalien added the bug label Sep 16, 2022
@myfunnyandy myfunnyandy added the managed on taiga This issue has been moved to our project at Taiga.io label Sep 19, 2022
@myfunnyandy
Copy link
Contributor

We've added the bug to our issue tracker: https://tree.taiga.io/project/penpot/issue/4130

We hope to fix it soon, thanks for such a detailed report.

@pabloalba pabloalba added this to the 1.17.0 milestone Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug managed on taiga This issue has been moved to our project at Taiga.io
Projects
None yet
Development

No branches or pull requests

4 participants