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

IED Editor switches back to first IED when starting to edit #496

Closed
dlabordus opened this issue Jan 19, 2022 · 3 comments · Fixed by #501
Closed

IED Editor switches back to first IED when starting to edit #496

dlabordus opened this issue Jan 19, 2022 · 3 comments · Fixed by #501
Assignees
Labels
Kind: Bug Something isn't working

Comments

@dlabordus
Copy link
Contributor

There is a bug that when you start to edit the selected IED in the IED Editor, the IED Editor always switches back to the first IED in the list.
This is caused by the rendering of the IED Editor where the first IED is always made selected.

@dlabordus dlabordus added Kind: Bug Something isn't working Priority: Urgent labels Jan 19, 2022
@dlabordus dlabordus self-assigned this Jan 19, 2022
@JakobVogelsang
Copy link
Collaborator

JakobVogelsang commented Jan 19, 2022

Yeah, when one test, it should not the first one to test with. Sorry I missed it. I think a quick solution could be to store the IED name to local Storage. If the IED name cannot be found to remove it and show the first IED.
It is however not really a clean solution. Here we have to have a closer look why we re-render the whole Editor every time a wizard-dialog opens or closes.

@dlabordus dlabordus added this to the Sprint 2022-02-04 milestone Jan 19, 2022
@dlabordus
Copy link
Contributor Author

dlabordus commented Jan 19, 2022

Damn, you triggered me in testing something. The same problem has the SLD now when selecting the Substation.

I though that saving the state better would fix this. But I need to think further...

@dlabordus
Copy link
Contributor Author

I found another solution by creating a variable outside the Plugin/LitElement and use that.
It also not perfect, because it now becomes a global variable, so I named it by the SLD "sldSelectedSubstation".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Kind: Bug Something isn't working
Projects
None yet
2 participants