Refactoring #1625

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

merge classes ILayout and ParticleLayout

Added by wuttke over 3 years ago. Updated over 3 years ago.

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

0%

Category:-
Target version:-

Description

or what other child of ILayout is waiting for implementation?

History

#1 Updated by herck over 3 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 over 3 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