kergomard

Results 141 comments of kergomard

Thank you very much for the PR @bidzanaaron ! Could you please move the whole multilayered if/else-construct to its own function and then use a single level of `if`s with...

Thank you very much @bidzanaaron ! This looks good as far as I can see, ...but will be hell to pick to newer versions. One thing: Could you please change...

Thank you very much @fhelfer ! We are getting there! Just one more thing: - Couldn't we instead of copying the code from ilTestImporter, move the first part of the...

Thank you very much @fhelfer for the update ! Merged and picked to release_9 and ported to ILIAS 10 and trunk.

@lukas-heinrich : You are aware that we are already having the same discussion here: https://github.com/ILIAS-eLearning/ILIAS/pull/7439 Temporarily assign this to you, but will change the assignment, if you keep this open.

Not really. I think we could actually close this one for the time being and use the linked PR for further discussions. The matter is not urgent, as there is...

Thank you very much for the PR @matheuszych ! I do not think this is correct as it also changes the overview table. There is a comment that there should...

Thank you very much for the PR @matheuszych ! Where does the magic number 13 come from? Could we tie it to something somewhere? This for me like it will...

Hi @matheuszych Thank you very much! I think we can mostly keep it as it is, but couldn't we bind this to a variable? What about setting it to `$il-font-size-base`?...

Hi @catenglaender I add you as a reviewer to this as there are changes in CSS and I assume we would like to keep the processes introduced by @Amstutz for...