sialcasa / mvvmFX

an Application Framework for implementing the MVVM Pattern with JavaFX
Apache License 2.0
494 stars 104 forks source link

Enhance general documentation of mvvmFx #412

Open mainrs opened 8 years ago

mainrs commented 8 years ago

There is nearly no Javadoc available for mvvmFx. Documentation would be really nice to have and would make life easier for new developers who want to try the framework out.

manuel-mauky commented 8 years ago

Hi Sven, I've looked through the code to see which parts are missing javadoc and I think your statement that "neary no javadoc" is an exaggeration ;-)

Can you point to actual classes/packages that need more documentation?

Here is an overview of our code and my opinion about documentation. For each class the percentage value shows how "complete" the javadoc is from my point of view. If something is missing I've added it in parentheses. Please tell me if you disagree on specific files.

Core module

de.saxsys.mvvmfx

de.saxsys.utils.commands

de.saxsys.utils.itemlist

de.saxsys.utils.mapping

de.saxsys.utils.notifications

de.saxsys.utils.validation

package-info.java missing

de.saxsys.utils.validation.visualization

viewlist

CDI

Guice

Testing Utils

Utils

de.saxsys.mvvmfx.utils.listener

de.saxsys.mvvmfx.utils.sizebinding

mainrs commented 8 years ago

Sorry for the inconvenience. I mostly was referring to the internal package. It might not be important on the first glance but documenting how the framework works under the hood would allow other people to get involved in a much simpler way.

manuel-mauky commented 8 years ago

Hi, I know that for example the logic in the FxmlViewLoader is not easy to understand. We have tried to document as much as possible but the whole process is still complex due to limitations of JavaFX' FXMLLoader. Can you point to classes or methods that need more documentation? Then I can try to add additional informations.

mainrs commented 8 years ago

Sure, will do later on...