Refactoring #1625

Feature #1290: === Core: framework ===

merge classes ILayout and ParticleLayout

Added by wuttke about 4 years ago. Updated about 4 years ago.

Status:RejectedStart date:04 Nov 2016
Priority:LowDue date:
Assignee:-% Done:


Target version:-


or what other child of ILayout is waiting for implementation?


#1 Updated by herck about 4 years ago

No need to merge. First of all, ILayout should define the interface for adding 'stuff' to layers or multilayers (open/closed). In this sense, reference to particles could be removed in ILayout, at the cost of a big refactoring of LayerStrategyBuilder.
Secondly, we will need a generic interface when we will implement domain structures.

#2 Updated by wuttke about 4 years ago

  • Status changed from Rfc to Rejected

OK, thanks. I'll improve the class comment, and add a "to consider" note about the possible refactoring.

Also available in: Atom PDF