lscharmer
lscharmer
Hi @klees, I saw that there is one question that I need to answer: > - [ ] to and either: If I'd use Transform::to on the result build by...
Hi @klees, I implemented the requested changes. With the change of: > - [ ] as: I think the creation of the parser would look a lot simpler if we'd...
Hi @klees, this PR is updated an can be reviewed again. Below are my answers to your review: > - [x] ProblemBuilder: You seem to be fixing some problem with...
Hi @klees, I updated this PR and addressed your points: > * [ ] CSS: Why do we need this transformation? Do we have a usecase for it? In agreement...
> @lscharmer Thanks for this PR. This is definitely needed. I would strongly suggest to extend the readme file https://github.com/ILIAS-eLearning/ILIAS/blob/trunk/components/ILIAS/Refinery/README.md with this PR, too, since this is what most devs...
> Hi @lscharmer and @oliversamoila, > > we have discussed via our UI-Coordinator channels. We would kindly ask you to rename this to `Breadcrumb Column` (instead of `Path` column...) because...
Hi @klees, thank you for your review. > * [x] examples: Please remove padding-left for the breadcrumb in tables. Since both the breadcrumb and the cell contain padding, we lose...
Hi @klees, thank you for your feedback. Just to be clear: With new syntax you mean the `@template*` annotations? The `Foo` syntax is not new, as we are using this...
> Hi @lscharmer would you mind to resolve the conflict please? Thanks a lot! Done
> Hi @lscharmer, here is also "Authority to (De-)Assign Authorities" missing in your comment :) Thx, Copy paste ... :)