5 Oct
2016
5 Oct
'16
12:35 p.m.
On 05 Oct 2016, at 12:08, Ecaterina Moraru (Valica)
<valicac(a)gmail.com> wrote:
On Wed, Oct 5, 2016 at 12:37 PM, Vincent Massol <vincent(a)massol.net> wrote:
However following this strategy you shouldn’t be allowed to blacklist the apps that you
blacklisted since they fail point A now (and you should consider them as extensions - we
bundle them but they could be not bundled, and the same problem will appear with contrib
extensions.
What I’m questioning is considering that Invitation, Panels, Scheduler, User Directory
and Tour are special applications and that we only want to hide those and all others
should be listed by default.
Hi devs,
With the recent introduction of the Applications Index (see
http://extensions.xwiki.org/xwiki/bin/view/Extension/
Application+Index+Application/) we need to agree on a few things.
In the past we had:
* We wanted all new app that you installed to automatically be visible in
the Applications Panel
* This is why the Applications Panel config had a blacklist (and not a
white list)
There are some nice side effects of the current behavior that I would not
want to lose:
A. after you install an app with the EM, it immediately appears (if it has
the UIX defined) in the AppBar and is ready for usage. The admin does not
need to do the second step manually. We save him a click (in practice is
more than a click).
B. after you create a new app with AWM, you automatically have it in the
AppBar. In theory that app should be useful for the user, since he manually
create it to his own liking. We are saving him a click+. The only problem is when the wiki is getting out of
control, too many apps,
no favorite order, just alphabetical. For this there is this issue
http://jira.xwiki.org/browse/XWIKI-13075
What we’ve done:
* We add the Applications Index
* We removed some apps from the Applications Panel. Namely: Invitation,
Panels, Scheduler, User Directory and Tour
applications. this was done
using hardcoded blacklist xobjects in PanelsCode.
ApplicationsPanelConfiguration.
The need:
* We need to remove this hack. It’s not normal for the Panels module to
know all the apps that shouldn’t be listed in it!
So the main problem is that currently the AppBar is defined in Panels
module, instead of the Application module. This is known and we also have
an issue for the move, see http://jira.xwiki.org/browse/XWIKI-13502 , we
just didn't had the time to do it. Proposal:
* Replace the blacklist configuration of the Applications Panel by a
whitelist one
* When a new app is installed, list it in the Applications Index but don’t
add it to the Applications Panel
* If an admin wants to add this new for his users, he’ll need to add it in
the Admin UI for the Applications Panel.
WDYT?
I'm not sure I like this very much :)
IMO the main problem is that we are using the "Add Application" UIX
http://platform.xwiki.org/xwiki/bin/view/ExtensionPoint/AddApplicationUIX
to define applications.
For me, ideally, we would have a separate mechanism to define an app (not
this UIX). Application Index would display all the XARs that contain an app
descritor, while the AppBar would list the apps that have the UIX. The problem with the AppBar and the blacklisting
appeared when we added the
UIX to applications that were not "user-oriented", not useful in the daily
flow, not needed for fast navigation prioritization, but that contained the
UIX since we added this step to our recommended development app practices.
Ideally the UIX should be added to applications intended for normal users
that need a fast access point in the AppBar.
This was catered for already. If you were not an admin the App Panel was only listing
user-oriented apps (there was a check for admins in apps). Personally I don’t agree with
the rationale that Admins are lesser users of the wiki and that they shouldn’t have fast
access to admin apps.
If an Admin installs a new admin app (for ex), that app will be displayed in the App Panel
and we certainly don’t want to add yet another blacklist xobject in the App Panel.
Having it as a white list, will reduce the
user-centered applications
discoverability and rapid access.
That’s the role of the App Index IMO. Maybe we need to make it more visible or maybe it’s
just because we’re not used to it yet.
So my understanding is that the behavior you’d like like is:
* Keep a blacklist of app for the XE flavor (KB flavor soon). So this means the config
should be moved to the flavor.
* All further apps installed by an admin will appear in the App Panel, even if they’re for
admins only. It’s up to the admin to decide to remove them or not and decide if he wants
to show the one we blacklisted in the flavor by default.
Sounds globally ok to me.
I guess the only point where I’m not 100% in agreement is the removal of quick access to
admin apps for admins but I’m not going to fight this since it’s not that important that
admin users have to do 2 extra clicks to access them, and if we consider the App Panel as
a favorite it’s acceptable.
Thanks
-Vincent
WDYT?
Thanks,
Caty
>
> Thanks
> -Vincent