-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: css_sort #190
feat: css_sort #190
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Conflicts: src/editor/mod.rs
Played around with this and the just merged |
@Hans-Maulwurf With the current state of this feature I'm quite happy. Does the description of it in the README sound helpful to you or do you have any improvement suggestions? |
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.
Will have to come back to review the code, let's start with the readme only.
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.
I think the code is completed and good! As usual minor naming and doc nits.
it screws with the sorting when other elements are involved under the same parent
Nice! 🎉 @EdJoPaTo could you give me feedback on my review(-style)? Was it helpful? Did you perceive me as aggressive or as friendly? Are there certain aspects I should focus more or less on? |
closes #189