Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apps-Management doesn't show missing dependencies #4581

Closed
georgehrke opened this issue Apr 28, 2017 · 3 comments
Closed

Apps-Management doesn't show missing dependencies #4581

georgehrke opened this issue Apr 28, 2017 · 3 comments
Labels
Milestone

Comments

@georgehrke
Copy link
Member

georgehrke commented Apr 28, 2017

Nextcloud 12 on the left, Nextcloud 11 on the right
apps - nextcloud chromium today at 2 18 43 pm

Nextcloud 12 only shows that you can't enable LDAP.
Nextcloud 11 gives you a reason why you can't enable LDAP.

I'd propose a tooltip on the enable button that shows missing dependencies

cc @nextcloud/designers

@georgehrke georgehrke added this to the Nextcloud 12.0 milestone Apr 28, 2017
@jancborchardt
Copy link
Member

I’d say it should be displayed directly, like the »New version available« message:
capture du 2017-04-29 17-47-21
(Cause a tooltip is quite hidden.)
Actually the <div class="warning"> in the <div class="actions"> can be used for that.

@georgehrke can you open a pull request for it?

@MorrisJobke
Copy link
Member

Let's fix this first in master and then backport it

@MorrisJobke
Copy link
Member

Fixed in #9565

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants