Code review comment for lp:~tapaal-contributor/tapaal/cpn-gui-dev

Revision history for this message
Kenneth Yrke Jørgensen (yrke) wrote (last edit ):

I noticed there seem to be a lot of TODO comments related to how colors should be handled.
I guess theses should all be adresse and can be removed safely?

If we leave in TODO comments for stuff that is already implemented or adresse elsewhere it can lead to confusion when we later revisit the code, as it might not be clear that the TODO comment is handled.

Intellij has a feature to list all todo/fixme etc. If there really is a todo, please add details about what is missing.

« Back to merge proposal