-
Notifications
You must be signed in to change notification settings - Fork 1
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
Saving collapse after reload #543
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eigentlich top, ich fänds noch gut, wenn der "Add Steps"-Button einen visuellen Indikator hätte, dass man ihn ein- und ausklappen kann, bzw auch in welchem State er sich befindet.
Ich finde die neuen Icons richtig gut! Wäre super, wenn wir die gleichen Icons auch für die neue UI beim Löschen von Runs verwenden könnten? :) @maximilianKalff In der Benutzung ist mir persönlich gerade aufgefallen, dass man als User nicht unbedingt alleine auf die Idee kommen würde, die Sections selbst einzuklappen. Vielleicht wäre noch die Option, alle Sections (außer die aktive natürlich) automatisch einzuklappen praktisch? Da wir ja auch noch andere Features für die Sidebar (Visualisierung, welche Steps bereits berechnet wurden) planen, könnten wir vielleicht über 1-2 Steuerelemente über der ersten Section nachdenken ... Aber das nur als Idee, ich finde deine Änderung super! 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Top, alles schick👍
Description
In order to make the editing of steps easier and prevent automatic collapsing of all sections except for the active one, this PR is adding code that tracks the current state of the accordion to the local storage.
Also changed the remove icon and right-aligned it
This is a possible version of how this could be done, please give feedback if you have a different idea.
Changes
sidebar_section.html
runs.js
Testing
test by going trough different workflows and using the sidebar
local storage can be seen by entering inspection mode via "Application"
PR checklist
Development
Mergeability
black
Code review