Would be useful to be able to quickly switch off Private Messages built-in UI especially when you want to implement your own using Views or something like that. Ofcourse that should only involve end-user part of UI.

Comments

crea’s picture

Title: Refactor code to move UI in submodule » Refactor code to move UI in a submodule

With UI that can be disabled, it will also become possible for other developers to provide their own versions of UI as third-party modules.

naheemsays’s picture

I think this is a bad idea - the module should be useable once enabled.

Giving too many options is also IMO a bad thing.

The module should be set up to give the people sane defaults.

If there is still a need to offer an alternative UI, it should be through means that do not damage the defaults.

crea’s picture

What makes UI in separate module not usable by default ? It's in many places like that already - Views has own Views UI, Rules has Rules UI etc. This is not new concept for users. Killswitches are good for developers unless you want other UI makers to constantly look at your code to modify their menu_alters() etc. Providing good defaults is a must, but providing it in separate module won't do any harm. Place a warning there, and you are done.
Eternal "do than undo" Drupal developer experience is often criticized and this is exactly the case.

berdir’s picture

You can't really compare Privatemsg and Views/Roles. For both of these projects, the UI is administration only, it's not needed once you're in production. However, for Privatemsg, the UI is for all users. Unless there is a full replacement for the UI, this isn't really useful imho.

You're welcome to provide patches, but I imagine that it could be rather hard to split the code. Not everything is yet covered by api functions and so on.

YK85’s picture

subscribing

robby.smith’s picture

subscribing

ptmkenny’s picture

Version: » 7.x-2.x-dev
Component: User interface » Code
Status: Active » Closed (won't fix)

Please feel free to re-open if you want to provide a patch for this.