Why not include page indicator? about mmm-pages HOT 3 CLOSED

Hello!

I primarily made it a separate module for modularity. I personally don't like it when I need to install some unrelated thing to use a single feature of it.

By splitting up the module into two, I allow people who want to use MMM-page-indicator for their purposes without requiring them to drag in MMM-pages.

MMM-pages is more versatile than you might think. People have used MMM-pages for personalized pages with face recognition, which doesn't make sense to include MMM-page-indicator.

from mmm-pages.

Aha, I see. It just seemed more natural to have a config option like: pageIndicator: true and not have to install and configure a separate module... I also thought you needed to have MMM-pages in order to use the indicator. Thanks for quick feedback.

from mmm-pages.

That's a cost that I've found acceptable. I don't see clean and (more importantly) expected way to ask people to install both.

The issue is that other modules need to add in support for MMM-page-indicators for it to properly work.

Please feel free to make another PR if you have another other suggestions! I'll close this issue for now, but feel free to ask to reopen.

from mmm-pages.