nbrugger-tgm / reactj

A reactive ui lib for easy MVC
LaTeX Project Public License v1.3c
10 stars 0 forks source link

Binding DSL #49

Closed nbrugger-tgm closed 2 years ago

nbrugger-tgm commented 3 years ago

Implementation of #42

nbrugger-tgm commented 3 years ago

Progress

The replacement of the old MVC system was started. Progress can be checked on feat/mvc-replace-dsl branch.

Blockages

While its technicaly very possibe to replace all old functionality with the new Binding DSL it is very cumbersome. The amount of code needed to do binder.biBind("property",component::setVal,component::getVal) is like ~3 times the characters.

Before the whole MVC system is replaced a solution should be found so that the MVC system doesn't needs to be rewritten a seccond time.

TODO

General Improvements

Minification

Or

sonarcloud[bot] commented 2 years ago

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

78.3% 78.3% Coverage
0.0% 0.0% Duplication

nbrugger-tgm commented 2 years ago

The Codecoverage report from SonarCloud is not correct, therefore a merge is fine.'

The codecoverage is 91 %