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

Icon annotation gives validation errors #1569

Closed
edwardalee opened this issue Jan 25, 2023 · 4 comments · Fixed by #1572
Closed

Icon annotation gives validation errors #1569

edwardalee opened this issue Jan 25, 2023 · 4 comments · Fixed by #1572
Assignees
Labels
bug Something isn't working validation

Comments

@edwardalee
Copy link
Collaborator

If you create the following LF file:

target C
@icon("Variables.png")
reactor Variables {}

and then compile it with lfc, you get:

1 | target C
2 | @icon("Variables.png")
  |       ^^^^^^^^^^^^^^^ File extension 'png"' is not supported. Provide any of: ico, png, jpeg, gif, bmp

Note that the trailing quotation mark is included. I tried to fix this, but it just led to another error in the same validation code.

@edwardalee edwardalee added bug Something isn't working validation labels Jan 25, 2023
@cmnrd
Copy link
Collaborator

cmnrd commented Jan 25, 2023

I think I know where the problem is. I will push a fix as soon as my build setup works again.

@lhstrh
Copy link
Member

lhstrh commented Jan 25, 2023

I vaguely recall this feature was quietly removed due to a refactoring. Perhaps @a-sr knows more...

cmnrd added a commit that referenced this issue Jan 26, 2023
@cmnrd
Copy link
Collaborator

cmnrd commented Jan 26, 2023

afaik the icon feature does not work at the moment. Nonetheless, the issue here is an actual bug in the validator.

@a-sr
Copy link
Collaborator

a-sr commented Jan 26, 2023

Icons work but only in Eclipse and not in VSCode (kieler/KLighD#146) and they will stop working as soon as #1482 is merged. I will reactivate them as soon as the required feature is implemented in Klighd.

cmnrd added a commit that referenced this issue Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working validation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants