mlem icon indicating copy to clipboard operation
mlem copied to clipboard

Haptic Refactoring

Open EricBAndrews opened this issue 2 years ago • 2 comments

Haptics should be moved to the edit middleware layer, rather than being scattered about as they currently are.

EricBAndrews avatar Aug 03 '23 00:08 EricBAndrews

@EricBAndrews - I think the haptics should be kept out of the repository layer as there may be places where we'd want to make a call to refresh something in the background, or chain a few calls together from one method on the repository which would then always result in haptics being fired.

I think we should keep the firing of haptics at the call site to give that flexibility?

mormaer avatar Aug 03 '23 08:08 mormaer

That's a good point. This feels like a perfect application for the missing final chunk of middleware that handles middleware models (something like a fleshed-out tracker object)

EricBAndrews avatar Aug 04 '23 03:08 EricBAndrews